Skip to content

chore(agent-designer): default AGENTS_API_ENABLED on with a kill switch#607

Merged
philmerrell merged 1 commit into
developfrom
chore/agents-api-default-on
Jul 9, 2026
Merged

chore(agent-designer): default AGENTS_API_ENABLED on with a kill switch#607
philmerrell merged 1 commit into
developfrom
chore/agents-api-default-on

Conversation

@philmerrell

Copy link
Copy Markdown
Contributor

The Agent Designer is complete, so flip AGENTS_API_ENABLED from opt-in to default ON with a kill switch, matching house style for shipped features (scheduled_runs, memorySpaces).

  • backend agents_enabled(): empty-string-safe default-on — unset/empty ⇒ enabled, only literal "false" disables.
  • CDK config.agents.enabled: mirror the memorySpaces/scheduledRuns ternary (!== 'false' + context ?? true) so an unset/empty GitHub Actions var can't silently disable it.
  • tests: added the Agents API default-on/empty/kill-switch/context suite to config.test.ts; renamed the app-api-environment threading test.

Scope note: this makes the /agents/* API available everywhere (stops the per-env 404), but the SPA nav stays preview-gated (system-admin + "Preview" badge), so it does not broaden user-facing exposure. Widening who sees Agents (preview cohort) is separate follow-up work.

Verified: infra tsc + 77 flag tests green; backend agents route + resolver tests green; default-on behavior probed (unset/empty→on, false→off, true→on).

🤖 Generated with Claude Code

The Agent Designer is complete (contract → surface → resolution → Designer UI →
binding reflection), so flip the feature flag from opt-in to default-on, matching the
house style for shipped features (scheduled_runs / memorySpaces).

- backend `agents_enabled()`: empty-string-safe default-on — unset/empty ⇒ enabled,
  only the literal "false" disables (was `== "true"`, default off).
- CDK `config.agents.enabled`: mirror the memorySpaces/scheduledRuns ternary
  (`!== 'false'` + context fallback `?? true`), so an unset/empty GitHub Actions var
  can't silently disable it.
- Tests: add the Agents API default-on/empty/kill-switch/context suite to config.test.ts
  (mirrors Memory Spaces); rename the app-api-environment threading test (no longer
  "default off").

The `/agents/*` API now ships everywhere; the SPA nav stays preview-gated (system-admin
+ "Preview" badge) until Assistants are deprecated, so this doesn't broaden user-facing
exposure — it just stops the API 404ing per-environment.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@philmerrell
philmerrell merged commit df5e754 into develop Jul 9, 2026
4 checks passed
@philmerrell
philmerrell deleted the chore/agents-api-default-on branch July 9, 2026 03:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant