Conversation
Chat-history PDF: shorten front matter to title, subtitle, and one date line so the first prompt lands on page one; move scope and coverage to the closing section; keep the existing entry style; phase labels stick to the next message and the footer grid keeps title and page number apart. Survey review: open with an "Assessment in brief" callout after Scope, label each approach by mechanism and best evidence, use horizontal-rule tables with repeating headers and content-fitted widths, write urgency as text, and close with a next step. Plain title line, 18 mm margins, 10 pt justified body; the placeholder now fits one page. Both templates accept --input heading-font for builds without system fonts. Add a render test for tables that cross page boundaries. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
report_table asserts that headers match columns and every row matches the column count, naming the offending row, instead of shifting cells silently. problem_table gains the same headers/columns overrides as compare_table; stage defaults its fill; the survey template defines the sans fallback tuple once. The chat template accepts --input body-font so a bundled-fonts build is warning-free, and a new test renders it that way and checks the first prompt lands on page one with scope and coverage in the closing section. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Member
Author
Self-review (fresh-context reviewer, commit 2319ff1)Git audit: GATE PASS. harm.secrets OK, harm.dangerous-files OK, harm.generated-artifacts OK, harm.cache OK, harm.debug-leftovers OK (only the survey scaffold's intentional Design: DRY minor issue (survey template repeated the Findings and disposition
Gate after fixes: |
Rename "What and why" to "Overview" and add a "Key questions" section between it and the techniques: one subsection per subtopic with the question, why it matters (understanding and/or practical value), and where it stands. Strengths and limitations remain for techniques only. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
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 free
to 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.
Observation
The chat-history PDF spent most of page one on front matter (large title, subtitle, date line, scope and coverage paragraphs, a Transcript heading) before the first prompt appeared. The survey scaffold used pastel rounded boxes, fully ruled tables with fixed column fractions, and two figures before the first approach, and its placeholder already ran to two pages.
Issue
Both reports exist to present information quickly. Front matter that pushes the content down, and decoration that does not carry information, work against that. The survey table widths also did not fit their own header and priority labels, and long tables did not repeat headers across pages.
Fix
skills/dump-chat-history/assets/report.typ: front matter is title, optional subtitle, one date line; the first prompt follows directly. Scope and coverage move to "Sources and limits". The per-message entry style is unchanged. Phase labels stick to the next message, and the footer grid keeps the title clear of the page number.skills/survey/template.typ: plain title line, 18 mm margins, 10 pt justified body, "Assessment in brief" callout after Scope, "Mechanism / Best evidence" labels per approach, a sharedreport-tablehelper with horizontal rules, repeating headers, and content-fitted widths;compare_tableaccepts customheadersandcolumns; urgency labels are written text; a "Next step" paragraph closes the report. The placeholder fits one page.skills/survey/template.typandskills/survey/SKILL.md: "What and why" becomes "Overview", and a new "Key questions" section sits before the techniques: one subsection per subtopic with the question, why it matters (understanding and/or practical value), and where it stands. Strengths and limitations stay with techniques only.--input heading-font=...(the chat template alsobody-font) for builds without system fonts.skills/dump-chat-history/SKILL.md,skills/survey/SKILL.md,skills/how-to-write-ideas-report/references/typst-reference.md: describe the resulting layout.Rendered comparison (committed / earlier redesign / this PR): https://claude.ai/artifact/VY1qV8uD11TSXteN1LAjqr
Verification
python3 scripts/validate_skills.py: 16 skills validated.python3 -m pytest -q: 256 passed, including the existingtest_pdf_handles_long_original_text_and_structured_answers(every original string survives the PDF) and the newtests/test_typst_reports.py(a 45-row problem table paginates with the header repeated on every page and all rows present;compare_tablewith custom headers and the legacy positionalstagecolor still compile without warnings under--ignore-system-fonts).🤖 Generated with Claude Code