Skip to content

docs: document doctor session-recovery diagnostic (#4539) - #4548

Merged
Hmbown merged 1 commit into
mainfrom
claude/issue-4542-20260719-0030
Jul 19, 2026
Merged

docs: document doctor session-recovery diagnostic (#4539)#4548
Hmbown merged 1 commit into
mainfrom
claude/issue-4542-20260719-0030

Conversation

@Hmbown

@Hmbown Hmbown commented Jul 19, 2026

Copy link
Copy Markdown
Owner

Summary

Adds one clarification paragraph to docs/GUIDE.md documenting the doctor session-recovery diagnostic added in #4539: read-only, filename-level comparison, six statuses, additive to doctor / doctor --json, explicit CODEWHALE_HOME suppresses ambient inspection.

Produced by the Claude issue worker on #4542 as the final end-to-end verification of #4537/#4543: exact-@claude maintainer gate, signed branch from main (b6a9ab0, signature verified), no auto-PR. This PR is the human-click step.

Closes #4542.

🤖 Generated with Claude Code

Add a short clarification paragraph after the existing doctor-check
section in docs/GUIDE.md describing the session-recovery diagnostic
added in #4539: a read-only, filename-level comparison of legacy vs.
current session state, additive to `doctor` / `doctor --json`, with
its migration_pending/migration_incomplete statuses pointing back to
the existing ~/.deepseek -> ~/.codewhale migration guidance.

Docs-only change, verifying #4542.

Co-authored-by: Hunter Bown <101357273+Hmbown@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 19, 2026 00:47
@cursor

cursor Bot commented Jul 19, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@claude

claude Bot commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Claude finished @Hmbown's task in 43s —— View job


Review: docs: document doctor session-recovery diagnostic (#4539)

Verdict: accurate, no changes needed.

I cross-checked the added paragraph in docs/GUIDE.md:123-130 against the actual DoctorSessionRecoveryStatus implementation in crates/tui/src/main.rs:

  • Six statuses: isolated, no_legacy_sessions, migration_pending, migration_incomplete, migration_complete, scan_failed — matches the enum at crates/tui/src/main.rs:4524-4530 exactly, including the as_str() string forms at 4536-4541.
  • Read-only, filename-level comparison: matches doctor_session_recovery_report (crates/tui/src/main.rs:4633-4736), which only calls read_dir/symlink_metadata on .json filenames and never opens file contents — confirmed by its own doc comment at 4626-4632.
  • Additive to doctor / doctor --json: confirmed — print_doctor_session_recovery_report (4800) and doctor_session_recovery_json (4895) are both called alongside the existing legacy-state report rather than replacing it.
  • Explicit CODEWHALE_HOME suppresses inspection: confirmed — codewhale_home_is_explicit short-circuits to Isolated before any directory scan (4652-4655).
  • The "same legacy-path migration described above" cross-reference correctly points back to the ~/.deepseek~/.codewhale migration text a few lines earlier in the same doc.

No correctness, provider-safety, reuse, test-coverage, or security concerns — this is a pure documentation addition (+9/-0) with no code changes, and every technical claim in the new paragraph is verifiably true against the implementation it documents.

@Hmbown
Hmbown merged commit b0a0efb into main Jul 19, 2026
23 checks passed
@Hmbown
Hmbown deleted the claude/issue-4542-20260719-0030 branch July 24, 2026 21:10
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.

test: verify Claude issue worker end-to-end (docs-only)

2 participants