Skip to content

nightly: workflow.md 107 tokens smaller - #3

Merged
github-actions[bot] merged 3 commits into
mainfrom
nightly/40786733141
Oct 28, 2026
Merged

github-actions[bot] merged 3 commits into
mainfrom
nightly/40786733141

Conversation

@zolotarev

Copy link
Copy Markdown
Owner

Nightly simplification (run 40786733141), opened by the maintainer because the repository does not yet allow Actions to create PRs.

  • templates/workflow.md: 3423 → 3316 tokens (−107); total prompt tokens 7085 → 6978
  • repeated/decorative wording removed; code unchanged
  • VERSION 12, CHANGELOG, budgets lowered

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

VERDICT: rework

  • templates/workflow.md (Self-improvement, "Collect" bullet): the newline before - **Review … was lost, so the text now reads …→ <what to change>".- **Review — main session…`. This breaks the list in the rendered template, which every agent reads. Restore the line break.
  • tools/budgets.json: code_lines is raised 5618 → 5620, but the PR says "code unchanged". A nightly simplification must not raise a budget. Revert it to 5618, or explain which code lines were added.
  • lib/dpcli/init.py:9: VERSION = 12 # bump… lost the two spaces before # that the original had. Restore them so the diff is only 11 → 12.
  • templates/workflow.md: the same bullet drops "No status here." This is the only change that removes a rule, and the CHANGELOG says "no rule changed". Restore it, or correct the CHANGELOG line.
  • The other trims (step 3 now points to "Acceptance", which does cover the merge → accept --here → remove-copy sequence; the merged search and docs bullets; the shortened Monitoring and Sessions lines) are fine and saved ~107 tokens. Recheck tools/budgets.json after the fixes.

github-actions Bot and others added 2 commits October 28, 2026 23:28
@github-actions

Copy link
Copy Markdown

VERDICT: accept

  • All four items from the earlier rework are fixed in the current diff: Self-improvement is untouched, code_lines stays 5618, and lib/dpcli/__init__.py:9 changes only 11 → 12.
  • templates/workflow.md: step 3 now points to "Acceptance", and that section (lines 57-65) has the merge → accept --here → remove-copy sequence, so no rule is lost. The search and docs bullets are merged without loss. This matches the CHANGELOG line "no rule changed".
  • tools/budgets.json: prompt_tokens 7085 → 6982 and prompt_max 3423 → 3320 are lowered. The PR body reports 6978 and 3316, so they are 4 tokens above the measured values. Nit: tighten them if the budget check allows.
  • VERSION is bumped and the CHANGELOG entry is added. The text is in English, and nothing in it is project-specific.
  • I did not run the tests or the budget check. I could not check CI status from this sandbox.

@github-actions

Copy link
Copy Markdown

VERDICT: accept

  • Docs-only trim of templates/workflow.md (−107 tokens, per the PR description), with no code change. Lowering prompt cost is what CLAUDE.md asks for.
  • No rule is lost. templates/workflow.md:17 now points to "Acceptance", and step 5 of that section (line 65) still has merge, accept <ID> --here --commit, event merged and removing the copy. The deleted search bullet is folded into "Documentation". The "harness:" note command is still in "Self-improvement" (line 129).
  • Process is complete: VERSION 11→12 in lib/dpcli/__init__.py, a CHANGELOG entry for 12, English, and no project data.
  • tools/budgets.json is lowered (7085→6982, 3423→3320), not raised. It sits only a few tokens above the claimed new sizes (6978 / 3316), so the budget check should pass. I did not run the test suite or the budget check myself. Confirm CI is green before merge.

@github-actions github-actions Bot removed the rework label Oct 28, 2026
@github-actions
github-actions Bot merged commit b9e1048 into main Oct 28, 2026
2 checks passed
@github-actions
github-actions Bot deleted the nightly/40786733141 branch October 28, 2026 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant