fix(ops): stop operator copy from doubling the article before offerName - #178
Conversation
The canonical offer name "The Website Correction" already starts with "The".
Five operator copy sites interpolated it directly after an article,
producing doubled-article text ("the The Website Correction", "The The
Website Correction") in kickoff, loom script, sharpness brief, sales-call
prep, and prospect follow-up output.
Drop the redundant article at those sites and add a check-product-truth
guard that fails any active operator copy source interpolating offerName
directly after "the".
Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (7)
📝 WalkthroughWalkthroughThe change corrects doubled-article wording in five generated offer-copy scripts. It also adds a ChangesOffer name copy
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This localized change removes duplicated article text from operator-facing copy and adds a guard against recurrence; no actionable merge-blocking risk remains. Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
…main by PR #80 (duplicate guard) (#234) The duplicate-PR guard (PR #80, merged 2026-08-19) is now live on main, preventing the same-fix duplication pattern at the CI level. The named duplicate pairs (#36/#44, #39/#49, #40/#52) are functionally superseded: their underlying fixes are on main via PRs #135, #145, and #178. The prior 2026-08-15 lane run had concluded the guard existed but was stuck unmerged; on 2026-08-19 it landed and the item is resolved at the root-cause level. Verification-only run (no product or test files touched): - PR #80 source commit 2091c7a and merge commit 0a9909b are ancestors of origin/main - scripts/test-pr-duplicates.mjs → 'test-pr-duplicates: ok' (exit 0) - Superseding PRs #135 (d4f3ef4), #145 (fc44b42), #178 (77f6922) all on main - Guard workflow runs on pull_request events; not a required status, so existing work never blocks Co-authored-by: minimax-vps <minimax-vps@nish3451.dev>
1 participant
What
The canonical offer name "The Website Correction" already starts with "The". Five operator copy sites interpolated
offerNamedirectly after an article, producing doubled-article text in operator-facing output:scripts/draft-client-kickoff.mjs— "Thanks for approving the ${config.offerName}..."scripts/draft-loom-recording-script.mjs— "...positions the ${config.offerName} as the obvious next step."scripts/draft-recording-sharpness-brief.mjs— "CTA: Offer the ${config.offerName} at..."scripts/draft-sales-call-prep.mjs— "The ${FOUNDER_PILOT.offerName} is exactly a..."scripts/draft-prospect-message.mjs— "...the exact one-page ${config.offerName} scope."Fix
Dropped the redundant article at all five sites and added a regression guard in
scripts/check-product-truth.mjs(part ofnpm run ci) that fails any active operator copy source interpolating the offer name directly after "the".Verification
node --checkon all changed filesnode scripts/check-product-truth.mjsexit 0 (guard active)npm testexit 0git diff --checkcleanRelated: #144 (identical unmerged prior attempt — this lands the same fix fresh on current main).
Summary by CodeRabbit