docs(instruction-placement): record empirical Cursor, Grok Build and Muse Code loader results - #5563
Conversation
…Muse Code loader results Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cd6b8ab076
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
…am pages Refs #4283 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Claude finished @kyle-sexton's task in 1m 24s —— View job Security review complete
No security issues found. This PR is documentation-only: it replaces a parked placeholder record in Instruction-surface lens applied. The diff removes a "parked" decision record that said No inline comments were needed. |
|
Claude finished @kyle-sexton's task in 3m 18s —— View job Code review complete
SummaryReviewed the diff ( The two prior Codex P1 findings on this PR (recipe not committed; no upstream pointers) are already resolved by the latest commit ( Cross-checked the new "Loader behavior of Cursor, Grok Build, and Muse Code" record in 1 finding posted inline:
No other correctness or maintainability issues found in the reviewed diff. |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Claude finished @kyle-sexton's task in 1m 6s —— View job Security review (
|
|
Claude finished @kyle-sexton's task in 3m 26s —— View job Code review complete
SummaryThis is the CI code-review lane ( The as-of-date fix that Claude's earlier review flagged is correctly applied: 2 findings posted inline, both in the newly-added content:
Cross-checked the rest of the new loader-behavior table in |
…echeck trigger grammar Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Claude finished @kyle-sexton's task in 1m 20s —— View job Security review complete
No security issues found. This PR is documentation-only. It replaces the parked "Install-dependent loader tests parked Instruction-surface lens applied. The diff removes a "parked" scoping note that read: I also checked the PR body's note about the tools rewriting their own state on first run No inline comments were needed. |
|
Claude finished @kyle-sexton's task in 2m 34s —— View job Code review complete
SummaryThis lane already has three prior review rounds on this PR from earlier commits. I scoped this pass to what's new at HEAD ( Both prior findings are resolved:
1 new finding posted inline (a cross-file inconsistency in content added by this PR that the earlier reviews' file-by-file passes didn't catch):
Also checked |
|
Claude finished @kyle-sexton's task in 52s —— View job Security review complete
No security issues found. This PR touches only documentation/reference content, confirmed by reading all five changed files
No code, no GitHub Actions workflow, no executable script, and no config affecting runtime Instruction-surface lens applied (per this repo's own review criteria, since the diff deletes This matches the conclusion of the four prior review passes on this PR (Codex and three earlier |
… env var in both files Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…oader-tests Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Claude finished @kyle-sexton's task in 2m 0s —— View job Security review complete
No security issues found. This PR is documentation-only. The full diff against the true merge base (verified via
No application code, no GitHub Actions workflow, no executable script, and no config affecting runtime behavior is in the diff — only prose, a markdown results table, and a version/changelog bump. Instruction-surface lens applied (the diff deletes a standing "do not install those tools from this checkout / do not add CI that assumes they are present" note in This is the fifth security-review pass on this PR (Codex once, Claude four times across earlier commits |
|
Claude finished @kyle-sexton's task in 1m 42s —— View job Code review complete
SummaryThis lane has five prior review rounds on this PR, the last at commit The one open finding from the prior round is resolved. The merge of PR #5285 did not leave anything dangling. I checked for references to the old "Install-dependent loader tests parked" section name/anchor across the repo — none exist outside a historical CHANGELOG entry describing a past version, which is expected to stay as-is. The current Version/changelog bump is internally consistent: No new findings. Cross-checked the loader-behavior claims in This matches the trend of the five prior review rounds on this PR (Codex + four Claude passes), each of which found the diff clean once the flagged issue from that round was fixed. |
|
Claude finished @kyle-sexton's task in 45s —— View job Code review complete
SummaryThis lane has five prior review rounds on this PR, the most recent at commit Skip gate applied — no new review surface. The only change since the last pass is
No new findings. All prior findings from this lane (recheck-trigger grammar, table-row conflation, the Grok env-var naming mismatch, the as-of date) were already resolved as of |
|
Claude finished @kyle-sexton's task in 42s —— View job Security review complete
No security issues found. This CI run is triggered by The PR's own changed files remain the same five, all documentation/reference content:
No application code, no GitHub Actions workflow, no executable script, and no config affecting Instruction-surface lens applied (the diff deletes a standing "do not install those tools This is the sixth security-review pass on this PR (Codex once, Claude five times across commits |
Refs: #4283
Summary
Cursor, Grok Build and Muse Code loader claims were graded docs/source because none of the tools had been run. Cursor (cursor-agent 2026.09.28-64d2043), Grok Build (1.0.41) and Muse Code (1.4.1) are now installed on the desktop host, and the loader recipe was run against each. This PR replaces the parked record with the observed results and lists the bullets that stay open, each with its reason.
Fix
plugins/instruction-placement/skills/migrate/reference/sources.md: the parked-install record is replaced by a four-part record (claim, basis with tool versions and recipe, as-of, recheck trigger) holding the per-tool results and the open bullets.docs/specs/agent-doc-surfaces.md: rows corrected where the observed behavior differs from the docs-grade claim.Open bullets: Cursor Team/Project/User precedence,
~/.cursor/ruleson disk and sync, Cursor editor-vs-CLI comparison, Grok path-only reminder text, Grok MAX_WALK_DEPTH=10, Muse user-rules path and Windows resolution. Each needs the editor, a Team plan, a Windows host or an observation not available headless.Refs, notCloses: the issue's acceptance criteria say no write to user-scope config. No probe wrote any file there, but the tools rewrote their own state on first run (~/.config/muse/settings.json, chezmoi-managed;~/.grok/config.toml;~/.cursorchats and projects entries). The owner decides whether that meets the criterion and whether the issue closes.Verification
cursor-agent -p,grok -pandgrok inspect, andmuse exec, from the root and from nested directories.git status --porcelainin the worktree was empty after the runs..work/4283/.Related
sources.mdsection as "pending owner decision"; resolve the overlap when merging the base and keep this record)🤖 Generated with Claude Code