Skip to content

[docs] ENG-3684 - Disclose the marker edit that scan now makes - #171

Merged
mariojgt merged 1 commit into
mainfrom
vespera/5a297d29
Aug 25, 2026
Merged

mariojgt merged 1 commit into
mainfrom
vespera/5a297d29

Conversation

@mariojgt

Copy link
Copy Markdown
Contributor

Summary

  • Disclose that scan may write the production marker into JSX root shells (before the post), not only the widget tag after a successful post.
  • Align AGENT-INSTALL.md and README.md with the #165 behavior so agents auditing the tarball see every source edit.
  • Fix the uninstall note that claimed __PATCHSTACK_PROD__ never lives in source.

Supersedes #166 (same intent, rebased onto current main after the credential-file doc updates).

Test plan

  • Diff AGENT-INSTALL.md command table + “Package and command behavior” against src/cli.ts marker pass (runProductionMarker / guide production-marker steps)
  • Confirm README CLI blurb + disclosure-widget section match the same two edits and opt-out via "widget": false
  • Spot-check uninstall steps still remove a source marker on JSX roots
  • Optional gate: node field-test/run.mjs --persona hostile --rounds 3 (🟠 AGENT-INSTALL change per MAINTAINING.md)

Made with Cursor

Agents treat an undisclosed source edit as misrepresentation; since scan
writes the production marker into JSX roots, the docs must say so.

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderbuds

coderbuds Bot commented Aug 25, 2026

Copy link
Copy Markdown

Documentation clearly details the new production marker edit.

🎯 Quality: 92% Elite · 📦 Size: Small

📈 This month: Your 75th PR — above team average · Averaging Excellent

See how your team is trending →

@mariojgt

Copy link
Copy Markdown
Contributor Author

/review

@mariojgt
mariojgt merged commit 0facc8f into main Aug 25, 2026
6 checks passed
@mariojgt
mariojgt deleted the vespera/5a297d29 branch August 25, 2026 08:39
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