feat(start-work): add auto_commit config option

[acamq]

Summary

• Adds start_work.auto_commit configuration option to allow users to disable the automatic commit step in the /start-work workflow
• When auto_commit is false, "STEP 8: COMMIT ATOMIC UNIT" is removed from the orchestrator reminder and step numbering is adjusted accordingly

Usage

Users can now disable auto-commit in their configuration file:

{
  "start_work": {
    "auto_commit": false
  }
}

Changes

File	Change
src/config/schema/start-work.ts	NEW - Zod schema for start_work config with auto_commit: boolean (default: true)
src/config/schema/oh-my-opencode-config.ts	Added start_work to root schema
src/hooks/atlas/types.ts	Added autoCommit to AtlasHookOptions
src/hooks/atlas/atlas-hook.ts	Forward autoCommit option to tool handlers
src/hooks/atlas/tool-execute-after.ts	Pass autoCommit to buildOrchestratorReminder
src/hooks/atlas/verification-reminders.ts	Conditionally render commit step with renumbering
src/plugin/hooks/create-continuation-hooks.ts	Pass config to createAtlasHook

Verification

• ✅ Typecheck passes
• ✅ Build passes
• ✅ 41 Atlas hook tests pass
• ✅ JSON schema generated with new start_work option

Resolves #2197

---

Summary by cubic

Adds a start_work.auto_commit config to let users turn off automatic commits in the /start-work workflow. When disabled, the orchestrator reminder hides the commit step and renumbers the next step.

• New Features

  • Added start_work.auto_commit (default true) to the root config and JSON schema.
  • Wired autoCommit through Atlas hook and tool-execute-after to verification reminders.
  • Commit step renders only when enabled; next step number adjusts.
  • Updated AGENTS.md to include start_work.

• Bug Fixes

  • Cleaned AGENTS.md by removing an accidental keystroke and duplicate schema entries.

^{Written for commit 5e726a2. Summary will update on new commits.}

[cubic-dev-ai]

2 issues found across 9 files

Confidence score: 5/5

• This PR looks safe to merge since the issues are documentation-only with low severity and no runtime impact
• The most notable issue is duplicated schema tree lines in src/config/AGENTS.md, which could confuse readers but won’t affect behavior
• Pay close attention to src/config/AGENTS.md - duplicated entries and a stray '7ZB|' keystroke in the docs

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="src/config/AGENTS.md">

<violation number="1" location="src/config/AGENTS.md:7">
P2: Accidental keystroke insertion '7ZB|' corrupts documentation text</violation>

<violation number="2" location="src/config/AGENTS.md:36">
P2: Duplicated entries in schema tree - lines 36-37 duplicate lines 34-35</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[acamq]

@cubic-dev-ai recheck please

[cubic-dev-ai]

@cubic-dev-ai recheck please

@acamq I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

No issues found across 9 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Auto-approved: Adds a configuration option to toggle auto-commit; standard plumbing through config schemas, hooks, and UI logic with no breaking changes or regressions identified.}

[FrancoStino]

"permission": {
    "bash": {
      "git add *": "deny",
      "git commit *": "deny",
      "git push *": "deny"
    }
  }

But I wonder if your implementation is useful for saving tokens and processes. It should eliminate unnecessary steps.