Skip to content

docs(migration): fix min session commands in activate/attach example - #1090

Merged
twitchyliquid64 merged 1 commit into
mainfrom
docs/fix-migrate-session-commands
Jul 29, 2026
Merged

docs(migration): fix min session commands in activate/attach example#1090
twitchyliquid64 merged 1 commit into
mainfrom
docs/fix-migrate-session-commands

Conversation

@jessie-minimal

@jessie-minimal jessie-minimal commented Jul 29, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Member

Summary

Testing

Checklist

  • Docs updated if behavior changed
  • BREAKING CHANGE: footer present if this is a breaking change

Note

Fix min session command examples in migration guide

Corrects command examples in migrate-to-minimal-session.md to include the required session subcommand. Updates min activate, min attach <session ID>, and min activate --attach to their correct forms with the session subcommand.

Macroscope summarized a0d60ec.

Summary by CodeRabbit

  • Documentation
    • Updated the migration guide to reflect the current interactive Sessions workflow.
    • Replaced legacy activation and attachment commands with the corresponding min session commands.

@coderabbitai

coderabbitai Bot commented Jul 29, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The migration guide updates its [session] examples from legacy session commands to the v0.5.0 min session subcommand syntax.

Changes

Session command documentation

Layer / File(s) Summary
Update migration commands
docs/guide/migrate-to-minimal-session.md		 Replaces min activate and min attach examples with min session activate and min session attach forms.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: twitchyliquid64

Poem

A rabbit hops through commands anew,
“min session” makes the workflow true.
Activate, attach, and go—
The guide now helps the sessions flow! 🐇

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description preserves the template but leaves Summary, Testing, and Checklist effectively unfilled. Add a real summary, include any testing or note if none was run, and complete the checklist items relevant to this docs change.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Title check ✅ Passed The title is concise, conventional, and accurately describes the command-name fix in the migration guide.

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

@twitchyliquid64
twitchyliquid64 merged commit e6cf884 into main Jul 29, 2026
28 checks passed
@twitchyliquid64
twitchyliquid64 deleted the docs/fix-migrate-session-commands branch July 29, 2026 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@twitchyliquid64 twitchyliquid64 twitchyliquid64 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jessie-minimal @twitchyliquid64