Uh oh!
There was an error while loading. Please reload this page.
Allow verbatim plan numbers only, positive framing, and tighter targets - #478
Closed
neoneye wants to merge 1 commit into
Closed
Allow verbatim plan numbers only, positive framing, and tighter targets#478neoneye wants to merge 1 commit into
neoneye wants to merge 1 commit into
Conversation
Replace number constraints with: "Use numbers only when the project context provides them directly — do not calculate, derive, or estimate figures." This closes the arithmetic-derivation loophole (HK$470M → HK$141M) from PR #475 while still allowing models to cite real plan numbers verbatim. Replace negative prohibition ("Do NOT include 'Controls ... vs.'") with positive framing. Tighten targets: consequences 2-3 sentences, options one sentence, review_lever one sentence (20-40 words). All sources (field descriptions and system prompt) consistent. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
neoneye
commented
Apr 1, 2026
MemberAuthor
Self-improve iteration — analysis 73Verdict: CONDITIONAL Wins:
Gaps:
The verbatim-numbers approach is the right one — just needs to be applied to options too, and the review_lever field description needs the template-lock fix. |
neoneye
commented
Apr 1, 2026
MemberAuthor
Closing — verbatim-numbers works for consequences but missing from options field. Will bundle with template-lock fix in next PR. |
5 tasks
neoneye added a commit
that referenced
this pull request
Apr 1, 2026
Replace vague "only cite numbers if the project context provides evidence for them" with precise "use numbers only when the project context provides them directly — do not calculate, derive, or estimate figures" in consequences field description and system prompt. This single change eliminated fabricated % claims from consequences in analysis 73 (#478). No other changes — isolating this variable. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Supersedes #471, #473, #475, #477.
Test plan
🤖 Generated with Claude Code