Skip to content

feat(service): toggle subagents with HANSI_SUBAGENTS - #32

Merged
TorstenDittmann merged 1 commit into
mainfrom
claude/project-thread-iuqk2z
Nov 4, 2026
Merged

TorstenDittmann merged 1 commit into
mainfrom
claude/project-thread-iuqk2z

Conversation

@TorstenDittmann

Copy link
Copy Markdown
Contributor

Requested by Torsten · project thread

The engine already had a subagents switch on its context (it records subagents.off and passes maxSubagents: 0), but the service never set it, so subagents were always on.

  • New env variable HANSI_SUBAGENTS: enabled (default) or disabled, validated in apps/service/src/env.ts.
  • index.ts passes it to the engine context for review and chat jobs.
  • README env table and env.test.ts cover it.

Default stays enabled, so no deployment config change is needed.

bun run lint, bun run check and bun test apps/service pass.


Generated by Claude Code

The engine already took a subagents switch on its context, but the
service never set it. HANSI_SUBAGENTS (enabled by default) now does.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01F3M7tvR2ysWQRTZhdspc2e
Co-authored-by: Torsten Dittmann <1759475+TorstenDittmann@users.noreply.github.com>
@TorstenDittmann TorstenDittmann self-assigned this Nov 4, 2026
@hansi-codes

hansi-codes Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

Adds the HANSI_SUBAGENTS environment setting, defaulting to enabled, and passes its value into the engine contexts used for review and chat jobs. The README and environment tests document and cover the setting.

Verdict New comments Fixed Still open
✅ Approved 0 0 0
📂 Walkthrough · 4
File Change
README.md Documents the HANSI_SUBAGENTS environment variable and its default.
apps/service/src/env.test.ts Tests the default, disabled value, and rejection of invalid values.
apps/service/src/env.ts Validates HANSI_SUBAGENTS as enabled or disabled, defaulting to enabled.
apps/service/src/index.ts Passes the setting through to the engine contexts.

Reviewed 1c90c89 · Details · Comment @hansi-codes review to re-run, or mention @hansi-codes with a question.

@hansi-codes hansi-codes Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Tier S · Looks good to merge. Summary

@TorstenDittmann
TorstenDittmann merged commit 6254ebd into main Nov 4, 2026
4 checks passed
@TorstenDittmann
TorstenDittmann deleted the claude/project-thread-iuqk2z branch November 4, 2026 07:28
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.

2 participants