feat(skill): plan talk and document decks and review Japanese wording - #394
Merged
Merged
Conversation
Slides written with the skill read as machine-made in Japanese: stock phrases, dashes, 「効く」, label-colon bullets, and pages that contradict each other. The skill now separates talk decks (question dividers, no answer up front) from document decks (answer first, assertion headlines), adds Japanese wording rules, and a review step with a text/flag extractor and a context-free reader. Rules are adapted from coji/natural-japanese, minorun365's slide-story and writing guide, carnot-tech/consulting-pptx-skill and tyroneross/pyramid-principle; licenses are recorded in sources.md. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0116eREiYfHTFLy5Eu3nwh5f
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.
Summary
Decks written with the slide skill read as machine-made in Japanese. The problems were stock phrases, dashes, 「効く」, 「ラベル:説明」 bullets, and pages that contradict each other. The skill now makes the agent decide first whether the deck is a talk or a document, then apply matching story and headline rules. It also adds Japanese wording rules and a review step before delivery.
Motivation
Several people have published slide skills for Japanese decks. After surveying them, the maintainer chose to combine these:
A real 60-page conference deck was rebuilt from scratch in both modes to test the rules. The findings from those builds and their fresh-eye reviews are folded back in. The largest lesson was that reviewers mostly caught facts that differ between pages, not awkward words, so the review step now includes a cross-page consistency check.
Changes
skill/SKILL.md: add a step to plan the story and the words (talk vs document; never invent facts; name headlinesHeadline). "Check and deliver" now runs the review. Add a fallback preview route via soffice and pdftoppm.skill/references/story-talk.md(new):skill/references/story-document.md(new):skill/references/writing-ja.md(new): a table of expressions to avoid, with fixes; deck-level habits; notation; rules for revising an existing deck.skill/references/review.md(new):skill/scripts/deck-text.mjs(new): prints the headline track, mode-specific metrics and wording flags, including speaker notes. It resolves@office-kit/pptxfrom the slide project.skill/references/tsx.md: add authoring pitfalls hit in practice, such as anchor values,Color,exactOptionalPropertyTypesand table fills, plus a page-number pattern.skill/references/sources.md(new): upstream commits and licenses (MIT and Apache-2.0).consulting-deck-designskill link to the new references.Testing
node --test packages/dev/test/skill-deck-text.test.mjs(new) builds a deck throughinitProjectandbuildDeck, runs the script from the project, and checks that:?ratio is reported--modeis rejectednpx oxfmt --check .andnpx oxlintare clean.packages/dev/test/authoring.test.mjs, which builds the tsx.md agenda example, passes. Its separate "initialized TSX typechecks" case fails in my local worktree on unmodifiedmaintoo, because module resolution fails from the temp directory, so it is unrelated to this change.Breaking changes
None. This only changes the agent skill documents; no package API changes, so there is no changeset.
Checklist
flagged it above.
represents real work that warrants a maintainer's review, and I am willing to
defend each line in review.
🤖 Generated with Claude Code
https://claude.ai/code/session_0116eREiYfHTFLy5Eu3nwh5f