Skip to content

Add ENABLE_CLAUDE_CODE_MODS feature flag - #976

Closed
lilly-luo wants to merge 1 commit into
mainfrom
add-enable-claude-code-mods-flag
Closed

lilly-luo wants to merge 1 commit into
mainfrom
add-enable-claude-code-mods-flag

Conversation

@lilly-luo

@lilly-luo lilly-luo commented Oct 3, 2026 •

Copy link
Copy Markdown
Collaborator

What

  • ENABLE_CLAUDE_CODE_MODS environment variable + a shared TRUTHY_ENV_VALUES set (1/true, case-insensitive) + a general constants.claude_code_mods_enabled() reader.
  • When the flag is set, every Claude Code launch exports CLAUDE_CODE_ENABLE_FUNCTION_HOOKS=1 (via agents/claude.py launch()), so Claude Mods (hooks modules) can load.

Why

First slice of making the smart-routing TUI richer with Claude Mods. The flag and its effect are harness-general — deliberately not tied to the smart-routing launch path, since mods may be used elsewhere. The mod that actually renders is stacked in #977.

Notes

  • CLAUDE_CODE_ENABLE_FUNCTION_HOOKS is the documented opt-in for function hooks. On builds where Claude's mods rollout is still gated off for the account, this sets everything up but the mod won't paint until the rollout reaches that account.

Test

  • uv run pytest tests/test_agent_claude.py::TestClaudeCodeModsFlag tests/test_agent_claude.py::TestMinimumVersion
  • just lint (ruff + ty)

This pull request and its description were written by Isaac.

@lilly-luo
lilly-luo force-pushed the add-enable-claude-code-mods-flag branch 3 times, most recently from fdebc9d to c4af896 Compare October 3, 2026 17:26
@lilly-luo
lilly-luo force-pushed the add-enable-claude-code-mods-flag branch 2 times, most recently from fbd5869 to 6ffb346 Compare October 3, 2026 17:57
@lilly-luo
lilly-luo added this pull request to stack #978 October 3, 2026 17:59
Introduce the ENABLE_CLAUDE_CODE_MODS environment variable, a shared
TRUTHY_ENV_VALUES set ("1"/"true", case-insensitive), and a general
claude_code_mods_enabled() reader in constants. Harness-general and not tied
to smart routing; nothing is gated on it in this change.

Co-authored-by: Isaac <no-reply@databricks.com>
@lilly-luo
lilly-luo force-pushed the add-enable-claude-code-mods-flag branch from 6ffb346 to 9d84069 Compare October 3, 2026 18:43
@lilly-luo lilly-luo closed this Oct 8, 2026
@lilly-luo
lilly-luo removed this pull request from stack #978 October 13, 2026 23:40
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