feat(peek): document the assisted-apply "re-verify" step

[harry-harish]

Summary

peek's supervised assisted-apply loop (control shield + input handoff + set_intent banner + published recipe) already ships. What it didn't yet spell out is the "…and re-verify" half of "apply this fix and re-verify." This adds that convention — guidance only, no new tool/permission/behavior.

• In-context guidance (SERVER_INSTRUCTIONS + execute_action + set_intent descriptions): after a mutating action, re-read the target with get_element_detail (its value should match what you intended) and get_page_view (no validation error should have appeared) before advancing the status banner; if it didn't take, stop and report via set_intent rather than blind-retrying.
• Published recipe (assisted-apply-with-supervision): a new "Verifying each applied step" section + a Step-3 pointer (+ updatedAt bump).
• Honest masking caveat (load-bearing): password/email/PII field values come back masked (the same masking the recorder applies), so those are verified by the absence of an error, not by value — consistent with the existing get_element_detail/get_page_view docs.

The loop stays client-driven (the agent orchestrates already-shipped read tools); no server-side loop primitive is added (that would be the autonomous-agent drift the project deliberately avoids). execute_action remains consent-gated + destructive-overridden + audit-logged.

Test Plan

• String/recipe-only change — no new logic. stdio-smoke asserts the 16 tool names/count (descriptions aren't asserted); full @peekdev/mcp suite green (366/1 skip) with peek-mcp built so the smoke test runs.
• peek-docs build validates the recipe frontmatter (description untouched at 153/160; updatedAt parses as a date); verify-recipes passes.
• typecheck + biome clean; changeset @peekdev/mcp patch.

Post-merge: peek-docs has no auto-deploy — the recipe change needs a manual Vercel redeploy.

Summary by CodeRabbit

• Documentation
  • Clarified step-by-step guidance for assisted apply workflows.
  • Added instructions to re-check changes after each action, confirm updates before moving forward, and stop if a step does not take.
  • Added notes for masked password, email, and PII fields, which should be verified by the absence of errors rather than exact value matching.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: c5c8a5d8-fb54-4afd-ba0a-d76cffd6f015

📥 Commits

Reviewing files that changed from the base of the PR and between 945f4f2 and 8c2a806.

📒 Files selected for processing (3)

• .changeset/peek-assisted-apply-reverify.md
• apps/peek-docs/src/content/recipes/assisted-apply-with-supervision.md
• packages/peek-mcp/src/mcp/server.ts

---

📝 Walkthrough

Walkthrough

Adds apply-and-re-verify guidance to the MCP server's SERVER_INSTRUCTIONS string and the execute_action/set_intent tool descriptions, mirrored in the assisted-apply recipe doc with a new "Verifying each applied step" section and a patch-level changeset entry.

Changes

Assisted-apply re-verify guidance

Layer / File(s)	Summary
MCP server instruction and tool description updates packages/peek-mcp/src/mcp/server.ts	SERVER_INSTRUCTIONS extended with post-mutation re-read, stop-on-failure, and masked-field (password/email/PII) verification guidance; execute_action description updated with the same; set_intent description updated to instruct "stopped — didn't take" status on failure.
Recipe doc and changeset apps/peek-docs/src/content/recipes/assisted-apply-with-supervision.md, .changeset/peek-assisted-apply-reverify.md	updatedAt date bumped, Step 3 banner guidance updated to reference re-read confirmation, new "Verifying each applied step" section added describing the full confirmation flow and masked-field behavior, and a patch changeset note added.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• Cubenest/rrweb-stack#63: Directly modifies execute_action and related tool descriptions in the same server.ts file.
• Cubenest/rrweb-stack#104: Adds get_element_detail and observation tooling that this PR's re-verify guidance explicitly depends on.
• Cubenest/rrweb-stack#118: Modifies the same SERVER_INSTRUCTIONS text to guide agent behavior.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 66.67% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes the main documentation change about the assisted-apply re-verify step.
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.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/peek-assisted-apply-reverify

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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