Skip to content

[caveman] Optimize instruction verbosity — asciicharts.md (2026-07-05)#43624

Merged
pelikhan merged 1 commit into
mainfrom
caveman-optimize-20260705-6f72d818c4eab05d
Jul 5, 2026
Merged

[caveman] Optimize instruction verbosity — asciicharts.md (2026-07-05)#43624
pelikhan merged 1 commit into
mainfrom
caveman-optimize-20260705-6f72d818c4eab05d

Conversation

@github-actions

@github-actions github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Summary

Condenses .github/aw/asciicharts.md instruction verbosity by ~65% without changing semantics. All rules and guidance are preserved; only presentation is tightened.

Changes

.github/aw/asciicharts.md — instruction token optimization

  • Collapsed ## Goal multi-bullet section into a single inline sentence.
  • Merged ## RULES 9-bullet list into 4 consolidated bullets; replaced multi-line Bad/Good code-block examples with inline backtick comparison.
  • Collapsed ## BEST GLYPHS four separate subsections into three inline sentences.
  • Renamed chart-type subheadings to include role annotation inline (Sparkline (best overall), Table + Trend (best for dashboards)).
  • Condensed ## SCALING from 5 bullets + prose to 2 bullets.
  • Renamed ## MOBILE RULE## MOBILE; collapsed list + prose to one sentence.
  • Removed standalone ## OUTPUT STYLE and ## PRIORITY sections; folded priority order into ## GOLDEN RULE as an inline chain.
  • Removed all decorative --- horizontal rules.

Impact

  • Semantic changes: None — all original constraints preserved.
  • Token reduction: ~65% reduction in instruction footprint.
  • Breaking changes: None.

Generated by PR Description Updater for #43624 · 27.2 AIC · ⌖ 5.83 AIC · ⊞ 4.7K ·

Collapse repetitive goal/rule lists and filler sections while
preserving all glyphs, chart-type examples, and layout rules.

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions github-actions Bot added automation documentation Improvements or additions to documentation prompt-quality labels Jul 5, 2026
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown
Contributor Author

Hey @github-actions 👋 — nice work on compressing the asciicharts.md instruction file! The caveman pass looks solid: the glyph tables, chart-type examples, and alignment rules are all intact, and the token reduction (~96→33 lines of prose) is meaningful.

One mechanical flag: no test files were touched in this diff. For a markdown instruction file this is expected — there is nothing to unit-test — but it prevents an automatic lgtm signal under the contribution checklist. If the project has any snapshot or golden-file tests for rendered prompt outputs from this instruction, it would be worth verifying they still pass after the restructuring.

If you would like to address the test coverage question, here is a ready-to-use prompt:

Check whether .github/aw/asciicharts.md is covered by any snapshot, golden-file, or integration tests in the repository.
If tests exist, run them to confirm the caveman-optimized version of the file still produces equivalent chart output.
If no tests exist, note that in a comment and confirm the change is documentation-only with no behavioral impact.

Generated by ✅ Contribution Check · 266.4 AIC · ⌖ 29.1 AIC · ⊞ 6.2K ·

@pelikhan pelikhan merged commit f37fbbc into main Jul 5, 2026
@pelikhan pelikhan deleted the caveman-optimize-20260705-6f72d818c4eab05d branch July 5, 2026 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation documentation Improvements or additions to documentation prompt-quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant