Add CI workflow and bump Sonnet to 4.6 #3

Merged
amoussa1229 merged 3 commits from chore/ci-workflow-sonnet-bump into main 2026-05-15 17:21:43 +00:00
amoussa1229 commented 2026-05-15 17:14:45 +00:00 (Migrated from github.com)

Summary

Two of the highest-ROI items from the retrospective:

  • CI workflow (fb5dbb2): standalone .github/workflows/ci.yaml running ruff check + ruff format check + pytest --collect-only. Org reusable workflows (Sea-Haven-Industries/.github) are SAM/CDK-focused and live on a different account, so a thin per-repo workflow is the right shape here. Closes the gap where the pre-push hook was the only line of defense.
  • Sonnet bump (3638d6b): models.CLAUDE_SONNET from claude-sonnet-4-20250514 (EOL 2026-06-15) to claude-sonnet-4-6. Centralized constant made it a one-line change. Smoke-tested via --route-only; deprecation warning gone, router behavior unchanged.
  • .DS_Store added to local .gitignore (also covered by the user-global gitignore now, but belt-and-suspenders so the warning stops appearing on this repo).

Test plan

  • ruff check . clean
  • ruff format --check . clean (12 files)
  • python3 run.py --route-only "What is the capital of France?" → done route, no claude-sonnet-4-20250514 deprecation warning emitted
  • CI workflow yaml validates by structure inspection — actions/checkout@v6 and actions/setup-python@v5 versions confirmed against feedback_action_versions.md
  • CI itself will validate on this PR — first run of the new workflow

Rationale notes

  • pytest --collect-only rather than full pytest because the golden-set test requires live ANTHROPIC_API_KEY + COMPOSIO_API_KEY (the test module skips cleanly without them). Full pytest stays a local pre-push step. If you want CI to run the live tests later, add the secrets to repo settings and flip --collect-only to -q.
  • Standalone workflow (not org-reusable) because Sea-Haven-Industries/.github/.github/workflows/ci-python-sam.yaml is built for SAM Lambdas with cross-account OIDC. The orchestrator has none of that infrastructure. Matching conventions where they apply (Python 3.12, ruff, action versions).

Rollback

git revert 3638d6b fb5dbb2 — both commits independently revertable.

## Summary Two of the highest-ROI items from the retrospective: - **CI workflow** (`fb5dbb2`): standalone `.github/workflows/ci.yaml` running ruff check + ruff format check + `pytest --collect-only`. Org reusable workflows (`Sea-Haven-Industries/.github`) are SAM/CDK-focused and live on a different account, so a thin per-repo workflow is the right shape here. Closes the gap where the pre-push hook was the only line of defense. - **Sonnet bump** (`3638d6b`): `models.CLAUDE_SONNET` from `claude-sonnet-4-20250514` (EOL 2026-06-15) to `claude-sonnet-4-6`. Centralized constant made it a one-line change. Smoke-tested via `--route-only`; deprecation warning gone, router behavior unchanged. - **.DS_Store** added to local `.gitignore` (also covered by the user-global gitignore now, but belt-and-suspenders so the warning stops appearing on this repo). ## Test plan - [x] `ruff check .` clean - [x] `ruff format --check .` clean (12 files) - [x] `python3 run.py --route-only "What is the capital of France?"` → `done` route, no `claude-sonnet-4-20250514` deprecation warning emitted - [x] CI workflow yaml validates by structure inspection — actions/checkout@v6 and actions/setup-python@v5 versions confirmed against `feedback_action_versions.md` - [x] CI itself will validate on this PR — first run of the new workflow ## Rationale notes - `pytest --collect-only` rather than full pytest because the golden-set test requires live `ANTHROPIC_API_KEY` + `COMPOSIO_API_KEY` (the test module skips cleanly without them). Full pytest stays a local pre-push step. If you want CI to run the live tests later, add the secrets to repo settings and flip `--collect-only` to `-q`. - Standalone workflow (not org-reusable) because `Sea-Haven-Industries/.github/.github/workflows/ci-python-sam.yaml` is built for SAM Lambdas with cross-account OIDC. The orchestrator has none of that infrastructure. Matching conventions where they apply (Python 3.12, ruff, action versions). ## Rollback `git revert 3638d6b fb5dbb2` — both commits independently revertable.
This repo is archived. You cannot comment on pull requests.
No description provided.