Skip to content

[docs] Remove the commit step from the agent guides - #6489

Merged
ashrafchowdury merged 5 commits into
mainfrom
docs/agent-guides-accuracy-pass
Sep 4, 2026
Merged

[docs] Remove the commit step from the agent guides#6489
ashrafchowdury merged 5 commits into
mainfrom
docs/agent-guides-accuracy-pass

Conversation

@ashrafchowdury

@ashrafchowdury ashrafchowdury commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Context

Nine docs pages walked readers through a Commit > New version step that no longer exists. The agent playground has no Commit button in its Configuration header, and agents have no Registry page in the sidebar, so one guide also sent readers somewhere they cannot go.

The trigger and schedule guides carried a related problem. Both spent a numbered step on which version an automation runs, and the automations concept page devoted a section to the same choice.

Changes

The commit step is gone. Where a step was only "click Commit", it is removed. Where a step did real work, the commit clause is dropped from it.

Before:

1. Click `Save`. The dialog closes and the playground header state reads `Draft`.
2. Click `Commit` in the **Configuration** header.
3. Pick `New version`, keep or edit the commit message, and click `Commit`.

After:

Then click `Save`. The dialog closes and the agent uses the new instructions
from its next message on.

The trailing ## Commit sections in the harness and subscription guides are removed outright, along with ## Commit before you schedule, ## Commit before you subscribe, and ## Commit a version.

Version selection is gone from the automation pages. The ### 3. Which version runs? step is removed from both trigger guides and the step that followed is renumbered, the overview sentence in the create guide drops its Version clause, and the concept page loses its "Each automation runs a version you choose" section. Picking a version is not a decision the reader makes while filling in the form.

Persistence mechanics are out too. An intermediate draft explained how the configuration saves itself, naming a status indicator and describing when a change becomes a version. Readers do not need that, so the guides now say what they get: click Save or Create, and the change applies.

Notes

Every claim was checked against origin/release/v0.114.5 rather than the local working tree. That mattered: GitButler had an unmerged branch applied locally that adds an agent version-history drawer, and an early draft of these docs described it. It is not on the release branch.

Verified on the release branch:

  • The agent config header passes autoSave, and AgentConfigHeader renders no commit button under it.
  • The Registry sidebar item is scoped workflowCategories: ["app"], so agents never show it.

No What to QA section. Nothing here is user-visible beyond the docs site.

The agent configuration auto-commits (#6126): there is no Commit button in the
Configuration header, and agents have no Registry page in the sidebar. The guides
still walked users through Commit > New version as a required step.

Replaces those instructions with the auto-save behaviour, removes the now-empty
commit sections from the automation, harness and subscription guides, and points
version history at the vN chip in the agent header instead of Registry.

Also corrects the trigger Version field, which defaults to the latest revision
rather than pinning one.
The trigger and schedule drawers' Version field lets an automation follow the
agent's newest version or stay on one you choose. The docs described it as
pin-only, and built the automations concept page on a draft-versus-committed
review point that auto-commit removed.

Rewrites the 'Which version runs?' step in both guides and reframes the concept
section around the choice, without documenting the picker's internals.
@vercel

vercel Bot commented Sep 2, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
agenta-documentation Ready Ready Preview Sep 2, 2026 12:14pm UTC

Request Review

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

📘 Docs preview

Status ✅ Ready
Preview https://pr-6489-agenta-docs-preview.mahmoud-637.workers.dev/docs
Inspect Actions run
Commit f85288bf5f0e238e19a622ac654007cab4f27db0

Pages changed in this pull request

This comment updates in place on every push.

@coderabbitai

coderabbitai Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Summary

Summary by CodeRabbit

  • Documentation
    • Updated agent configuration guides to describe the streamlined Save workflow.
    • Clarified that schedules can use the newest agent version or a specifically selected version.
    • Removed outdated commit steps from automation, trigger, skill, MCP server, model, and subscription guides.
    • Simplified automation setup and updated guidance on permissions, version selection, and subscription-based models.

Walkthrough

The documentation removes manual agent commit steps, describes direct configuration saves, and updates automation version guidance.

Changes

Agent versioning documentation

Layer / File(s) Summary
Versioning concepts
docs/docs/concepts/01-agents.mdx
The concepts documentation describes configuration changes as versions without committed-state wording.
Configuration workflows
docs/docs/guides/01-write-your-agents-instructions.mdx, docs/docs/guides/03-manage-skills.mdx, docs/docs/guides/04-add-an-mcp-server.mdx, docs/docs/guides/06-create-an-automation.mdx
The guides replace manual Commit steps with direct Save or Create actions.
Automation version selection
docs/docs/guides/07-schedule-an-automation.mdx, docs/docs/guides/08-trigger-an-automation-from-an-app.mdx
The guides update schedule version choices and remove the app-trigger version-selection step.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: 🟡 Moderate · up to f8528

The guides now omit required commit and pinned-version steps for unattended automations, which can lead users to configure workflows with unclear authorization and execution-revision behavior; another save workflow still does not consistently explain when changes take effect. These documentation errors should be corrected before merging.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The description clearly explains the removal of obsolete commit instructions and related automation version-selection guidance from the documentation.
Title check ✅ Passed The title is concise, specific, and accurately identifies the primary change: removing the commit step from the agent guides.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (3 skipped: 3 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/agent-guides-accuracy-pass

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

The guides explained how the configuration persists itself: a state dot, a version
per save, a grant landing in the next version. That is internal behaviour, and it
dates the copy to one implementation.

Says what the reader gets instead. Click Save or Create, and the change applies.
@ashrafchowdury
ashrafchowdury changed the base branch from release/v0.114.5 to main September 2, 2026 12:04
@ashrafchowdury ashrafchowdury changed the title [docs] Bring the agent guides in line with auto-commit [docs] Remove the commit step from the agent guides Sep 2, 2026
Which version a schedule or trigger runs is not a decision the reader makes while
filling in the form, so the step was noise between the cadence and the message.

Removes it from both guides, renumbers the step that followed, and takes version
out of the schedule guide's description.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1


ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Team

Run ID: 0e46c662-1a9d-484a-bbcf-4b318853c1e0

📥 Commits

Reviewing files that changed from the base of the PR and between 742fce9 and b34647d.

📒 Files selected for processing (5)
  • docs/docs/concepts/01-agents.mdx
  • docs/docs/guides/01-write-your-agents-instructions.mdx
  • docs/docs/guides/03-manage-skills.mdx
  • docs/docs/guides/04-add-an-mcp-server.mdx
  • docs/docs/guides/06-create-an-automation.mdx
🚧 Files skipped from review as they are similar to previous changes (3)
  • docs/docs/concepts/01-agents.mdx
  • docs/docs/guides/01-write-your-agents-instructions.mdx
  • docs/docs/guides/03-manage-skills.mdx

Included review availability: Your plan provides up to 8 included reviews per hour; 6 remain after this review.

Comment thread docs/docs/guides/04-add-an-mcp-server.mdx
The trigger guides no longer walk through the Version field, so the overview
sentence and the concept section explaining the choice had nothing left to point
at.

Drops the Version clause from the create-an-automation overview and removes the
'Each automation runs a version you choose' section.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 2


ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Team

Run ID: 626d5c8c-5771-449f-8618-de1cbec99ea1

📥 Commits

Reviewing files that changed from the base of the PR and between b34647d and f85288b.

📒 Files selected for processing (4)
  • docs/docs/concepts/07-automations.mdx
  • docs/docs/guides/06-create-an-automation.mdx
  • docs/docs/guides/07-schedule-an-automation.mdx
  • docs/docs/guides/08-trigger-an-automation-from-an-app.mdx
💤 Files with no reviewable changes (1)
  • docs/docs/concepts/07-automations.mdx
🚧 Files skipped from review as they are similar to previous changes (1)
  • docs/docs/guides/07-schedule-an-automation.mdx

Included review availability: Your plan provides up to 8 included reviews per hour; 5 remain after this review.

Comment thread docs/docs/guides/06-create-an-automation.mdx
Comment thread docs/docs/guides/08-trigger-an-automation-from-an-app.mdx
@ashrafchowdury
ashrafchowdury changed the base branch from main to release/v0.114.7 September 3, 2026 06:01
@ashrafchowdury
ashrafchowdury changed the base branch from release/v0.114.7 to main September 3, 2026 12:04
@ashrafchowdury
ashrafchowdury merged commit 6461d38 into main Sep 4, 2026
15 checks passed
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