@laitszkin/apollo-toolkit 3.6.5 → 3.8.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/AGENTS.md +1 -0
- package/CHANGELOG.md +15 -0
- package/README.md +1 -0
- package/align-project-documents/SKILL.md +0 -7
- package/analyse-app-logs/SKILL.md +0 -4
- package/analyse-app-logs/scripts/__pycache__/filter_logs_by_time.cpython-312.pyc +0 -0
- package/analyse-app-logs/scripts/__pycache__/log_cli_utils.cpython-312.pyc +0 -0
- package/analyse-app-logs/scripts/__pycache__/search_logs.cpython-312.pyc +0 -0
- package/answering-questions-with-research/SKILL.md +0 -7
- package/archive-specs/SKILL.md +4 -15
- package/cjk-pdf/SKILL.md +68 -0
- package/codex/learn-skill-from-conversations/SKILL.md +0 -4
- package/commit-and-push/SKILL.md +0 -4
- package/deep-research-topics/SKILL.md +0 -4
- package/discover-edge-cases/SKILL.md +0 -4
- package/docs-to-voice/SKILL.md +0 -23
- package/docs-to-voice/scripts/__pycache__/docs_to_voice.cpython-312.pyc +0 -0
- package/enhance-existing-features/SKILL.md +0 -4
- package/financial-research/SKILL.md +2 -13
- package/generate-spec/SKILL.md +6 -10
- package/generate-spec/scripts/__pycache__/create-specscpython-312.pyc +0 -0
- package/implement-specs/SKILL.md +8 -56
- package/implement-specs-with-worktree/SKILL.md +18 -58
- package/katex/scripts/__pycache__/render_katex.cpython-312.pyc +0 -0
- package/merge-changes-from-local-branches/SKILL.md +3 -19
- package/merge-conflict-resolver/SKILL.md +46 -0
- package/open-github-issue/SKILL.md +2 -43
- package/open-github-issue/references/issue-schemas.md +54 -0
- package/open-github-issue/scripts/__pycache__/open_github_issue.cpython-312.pyc +0 -0
- package/open-source-pr-workflow/SKILL.md +0 -4
- package/openai-text-to-image-storyboard/SKILL.md +0 -12
- package/package.json +1 -1
- package/read-github-issue/SKILL.md +0 -4
- package/read-github-issue/scripts/__pycache__/find_issues.cpython-312.pyc +0 -0
- package/read-github-issue/scripts/__pycache__/read_issue.cpython-312.pyc +0 -0
- package/record-spending/SKILL.md +0 -4
- package/recover-missing-plan/SKILL.md +1 -6
- package/resolve-review-comments/SKILL.md +0 -11
- package/resolve-review-comments/scripts/__pycache__/review_threads.cpython-312.pyc +0 -0
- package/review-change-set/SKILL.md +0 -4
- package/review-codebases/SKILL.md +0 -12
- package/scheduled-runtime-health-check/SKILL.md +1 -20
- package/scheduled-runtime-health-check/references/output-format.md +20 -0
- package/ship-github-issue-fix/SKILL.md +0 -4
- package/solve-issues-found-during-review/CHANGELOG.md +9 -0
- package/solve-issues-found-during-review/README.md +16 -0
- package/solve-issues-found-during-review/SKILL.md +90 -0
- package/solve-issues-found-during-review/agents/openai.yaml +4 -0
- package/submission-readiness-check/SKILL.md +0 -4
- package/systematic-debug/SKILL.md +0 -7
- package/test-case-strategy/SKILL.md +0 -7
- package/text-to-short-video/SKILL.md +0 -20
- package/text-to-short-video/scripts/__pycache__/enforce_video_aspect_ratio.cpython-312.pyc +0 -0
- package/version-release/SKILL.md +86 -81
- package/weekly-financial-event-report/SKILL.md +2 -19
package/AGENTS.md
CHANGED
|
@@ -58,6 +58,7 @@ This repository enables users to install and run a curated set of reusable agent
|
|
|
58
58
|
- Users can schedule a bounded project runtime window, stop it automatically, and analyze module health from captured logs.
|
|
59
59
|
- Users can investigate gated or shadow LLM APIs by capturing real client request shapes, replaying verified traffic patterns, and attributing the likely underlying model through black-box fingerprinting.
|
|
60
60
|
- Users can build and maintain Solana programs and Rust clients using official Solana development workflows.
|
|
61
|
+
- Users can fix issues discovered during a review pass by processing findings from highest to lowest severity, with per-fix validation and full-scope re-validation.
|
|
61
62
|
- Users can add focused observability to opaque workflows through targeted logs, metrics, traces, and tests.
|
|
62
63
|
- Users can iteratively improve repository code quality through behavior-neutral naming, simplification, module-boundary, logging, and test-coverage passes.
|
|
63
64
|
- Users can iteratively improve repository performance through evidence-backed module scans, safe hot-path optimization, benchmark guardrails, batching, caching, allocation, concurrency, and repeated full-codebase stage gates.
|
package/CHANGELOG.md
CHANGED
|
@@ -2,11 +2,26 @@
|
|
|
2
2
|
|
|
3
3
|
All notable changes to this repository are documented in this file.
|
|
4
4
|
|
|
5
|
+
## [v3.7.0] - 2026-04-29
|
|
6
|
+
|
|
7
|
+
### Added
|
|
8
|
+
- Extract shared CJK PDF layout, merge-conflict-resolver, and common git-submission workflows into standalone reusable skills (`cjk-pdf`, `merge-conflict-resolver`)
|
|
9
|
+
- Establish skill-dependency pattern: `implement-specs-with-worktree` depends on `implement-specs`; `version-release` depends on `commit-and-push`
|
|
10
|
+
|
|
11
|
+
### Changed
|
|
12
|
+
- Strip ~375 lines of verbosity across 32 skill files: remove empty Dependencies sections, merge redundant Overview paragraphs, deduplicate repeated principles in `generate-spec`, and consolidate `archive-specs` workflow steps
|
|
13
|
+
- Extract inline output templates and issue schemas into skill-local reference files for `scheduled-runtime-health-check` and `open-github-issue`
|
|
14
|
+
|
|
5
15
|
## [Unreleased]
|
|
6
16
|
|
|
7
17
|
### Added
|
|
8
18
|
- (None yet)
|
|
9
19
|
|
|
20
|
+
## [v3.8.0] - 2026-04-30
|
|
21
|
+
|
|
22
|
+
### Added
|
|
23
|
+
- Add `solve-issues-found-during-review` skill: fix review findings from highest to lowest severity with per-fix validation and full-scope re-validation
|
|
24
|
+
|
|
10
25
|
## [v3.6.5] - 2026-04-29
|
|
11
26
|
|
|
12
27
|
### Fixed
|
package/README.md
CHANGED
|
@@ -49,6 +49,7 @@ A curated skill catalog for Codex, OpenClaw, Trae, Agents, and Claude Code with
|
|
|
49
49
|
- scheduled-runtime-health-check
|
|
50
50
|
- shadow-api-model-research
|
|
51
51
|
- solana-development
|
|
52
|
+
- solve-issues-found-during-review
|
|
52
53
|
- submission-readiness-check
|
|
53
54
|
- systematic-debug
|
|
54
55
|
- test-case-strategy
|
|
@@ -5,13 +5,6 @@ description: Read and understand a software project, then generate or align proj
|
|
|
5
5
|
|
|
6
6
|
# Align Project Documents
|
|
7
7
|
|
|
8
|
-
## Dependencies
|
|
9
|
-
|
|
10
|
-
- Required: none.
|
|
11
|
-
- Conditional: none.
|
|
12
|
-
- Optional: none.
|
|
13
|
-
- Fallback: not applicable.
|
|
14
|
-
|
|
15
8
|
## Standards
|
|
16
9
|
|
|
17
10
|
- Evidence: Treat source code, configuration, scripts, and tests as the source of truth.
|
|
@@ -19,10 +19,6 @@ description: Comprehensive application log investigation workflow that reads log
|
|
|
19
19
|
- Quality: Separate confirmed issues from hypotheses and include time-window, log, code, impact, and confidence evidence for each report.
|
|
20
20
|
- Output: Return incident summary, confirmed issues, hypotheses, monitoring improvements, and publication status.
|
|
21
21
|
|
|
22
|
-
## Overview
|
|
23
|
-
|
|
24
|
-
Use this skill to analyze application logs systematically with the codebase and runtime context, then report confirmed issues with clear evidence, confidence, and next actions.
|
|
25
|
-
|
|
26
22
|
## Core principles
|
|
27
23
|
|
|
28
24
|
- Prioritize evidence over assumptions; avoid speculative conclusions.
|
|
Binary file
|
|
Binary file
|
|
Binary file
|
|
@@ -5,13 +5,6 @@ description: This skill should be used when the user asks a question that needs
|
|
|
5
5
|
|
|
6
6
|
# Answering Questions With Research
|
|
7
7
|
|
|
8
|
-
## Dependencies
|
|
9
|
-
|
|
10
|
-
- Required: none.
|
|
11
|
-
- Conditional: none.
|
|
12
|
-
- Optional: none.
|
|
13
|
-
- Fallback: not applicable.
|
|
14
|
-
|
|
15
8
|
## Standards
|
|
16
9
|
|
|
17
10
|
- Evidence: Verify repository facts from local files first and confirm time-sensitive claims with current external sources.
|
package/archive-specs/SKILL.md
CHANGED
|
@@ -9,8 +9,6 @@ description: Convert completed project plan sets into maintainable project docum
|
|
|
9
9
|
|
|
10
10
|
- Required: `align-project-documents` to align repository docs with current code before archiving, and `maintain-project-constraints` to synchronize `AGENTS.md/CLAUDE.md` after the doc update.
|
|
11
11
|
- Conditional: none.
|
|
12
|
-
- Optional: none.
|
|
13
|
-
- Fallback: not applicable.
|
|
14
12
|
|
|
15
13
|
## Standards
|
|
16
14
|
|
|
@@ -58,19 +56,10 @@ Convert completed planning artifacts into stable, standardized project documenta
|
|
|
58
56
|
- `docs/architecture.md`
|
|
59
57
|
- `docs/features.md`
|
|
60
58
|
- `docs/developer-guide.md`
|
|
59
|
+
- Ensure the categorized docs cover: installation and deployment, configuration, external services with required credentials and API key acquisition tutorials when applicable, architecture and module boundaries, feature introductions, and developer onboarding context.
|
|
61
60
|
- If the repository already uses different doc paths, preserve the established locations only when the resulting documents still match the same categorized sections and remain easy to maintain.
|
|
62
61
|
|
|
63
|
-
### 4)
|
|
64
|
-
|
|
65
|
-
Ensure the split project docs cover all of the following:
|
|
66
|
-
- how to install and deploy the project
|
|
67
|
-
- how to configure the project
|
|
68
|
-
- external services, required credentials, and API key acquisition tutorials when applicable
|
|
69
|
-
- project architecture and key module boundaries
|
|
70
|
-
- project feature introductions and user-facing flows
|
|
71
|
-
- project context developers should understand before making changes
|
|
72
|
-
|
|
73
|
-
### 5) Write the configuration and external-service guidance carefully
|
|
62
|
+
### 4) Write the configuration and external-service guidance carefully
|
|
74
63
|
|
|
75
64
|
- List each env var, config file, or secret only when supported by repository evidence.
|
|
76
65
|
- For every external service, document:
|
|
@@ -81,7 +70,7 @@ Ensure the split project docs cover all of the following:
|
|
|
81
70
|
- any safe-development notes such as sandbox/test-mode usage
|
|
82
71
|
- If the repository does not show how to obtain a credential, say so explicitly and point to the service's official setup page rather than guessing steps.
|
|
83
72
|
|
|
84
|
-
###
|
|
73
|
+
### 5) Keep README short and the doc set navigable
|
|
85
74
|
|
|
86
75
|
- `README.md` should stay short: project intro, quick install/deploy, major features, and key doc links.
|
|
87
76
|
- `docs/README.md` should act as the reference list for the categorized docs.
|
|
@@ -89,7 +78,7 @@ Ensure the split project docs cover all of the following:
|
|
|
89
78
|
- Remove template placeholders and stale planning language before finishing.
|
|
90
79
|
- After the docs are aligned, run `maintain-project-constraints` whenever the documentation changes imply `AGENTS.md/CLAUDE.md` needs to reflect updated workflows, commands, or repository capabilities.
|
|
91
80
|
|
|
92
|
-
###
|
|
81
|
+
### 6) Archive superseded spec files after successful conversion
|
|
93
82
|
|
|
94
83
|
- After the standardized project docs are complete and verified, archive the old source spec files that were converted.
|
|
95
84
|
- Prefer moving the consumed `spec.md`, `tasks.md`, `checklist.md`, `contract.md`, `design.md`, and when applicable their shared `coordination.md`, or the containing batch/spec plan directory, into a clearly marked archive path instead of leaving them mixed with active docs.
|
package/cjk-pdf/SKILL.md
ADDED
|
@@ -0,0 +1,68 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: cjk-pdf
|
|
3
|
+
description: Handle CJK PDF font selection, rendering requirements, and visual QA for Chinese-compatible PDF output. Use when any skill needs to render PDFs with Chinese or mixed CJK text, verify font paths, apply content safety rules, and perform screenshot-based visual QA.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# CJK PDF
|
|
7
|
+
|
|
8
|
+
## Dependencies
|
|
9
|
+
|
|
10
|
+
- Required: `pdf` for all PDF rendering and visual QA.
|
|
11
|
+
- Conditional: none.
|
|
12
|
+
- Optional: none.
|
|
13
|
+
- Fallback: If `pdf` is unavailable, stop and report the missing dependency.
|
|
14
|
+
|
|
15
|
+
## Standards
|
|
16
|
+
|
|
17
|
+
- Evidence: Verify font paths exist on the current macOS host before rendering.
|
|
18
|
+
- Execution: Select fonts → enforce content safety → delegate to `pdf` → screenshot QA → cleanup.
|
|
19
|
+
- Quality: Verify CJK glyphs render correctly (no tofu boxes), headings and body text are balanced, tables stay readable.
|
|
20
|
+
- Output: Return the final verified PDF path after passing visual QA.
|
|
21
|
+
|
|
22
|
+
## Workflow
|
|
23
|
+
|
|
24
|
+
### 1) Select CJK fonts (macOS)
|
|
25
|
+
|
|
26
|
+
Prefer fonts in this order (verify path exists before use):
|
|
27
|
+
1. `/System/Library/Fonts/Hiragino Sans GB.ttc`
|
|
28
|
+
2. `/System/Library/Fonts/Supplemental/Songti.ttc`
|
|
29
|
+
3. `/Library/Fonts/Arial Unicode.ttf`
|
|
30
|
+
4. `/System/Library/Fonts/STHeiti Medium.ttc`
|
|
31
|
+
5. `/System/Library/Fonts/STHeiti Light.ttc`
|
|
32
|
+
|
|
33
|
+
- Do not hardcode missing or poorly-rendering fonts.
|
|
34
|
+
- Do not assume `PingFang` is available on every macOS host.
|
|
35
|
+
- If the `pdf` skill already has a verified CJK-safe default, reuse it instead of overriding.
|
|
36
|
+
|
|
37
|
+
### 2) Content safety
|
|
38
|
+
|
|
39
|
+
- Use Chinese-compatible characters, punctuation, and fonts.
|
|
40
|
+
- Avoid emoji, decorative symbols, and unusual glyphs that often break rendering.
|
|
41
|
+
|
|
42
|
+
### 3) Delegate rendering to `pdf`
|
|
43
|
+
|
|
44
|
+
- Pass content and font requirements to the `pdf` skill.
|
|
45
|
+
- For tables with long Chinese phrases: require wrapped paragraph cells, width-constrained columns, and row heights that expand with content.
|
|
46
|
+
|
|
47
|
+
### 4) Visual QA
|
|
48
|
+
|
|
49
|
+
Open the rendered PDF and inspect:
|
|
50
|
+
- First page — layout, title, headings
|
|
51
|
+
- One page with a table or structured data
|
|
52
|
+
- One page with dense paragraph text
|
|
53
|
+
|
|
54
|
+
Verify:
|
|
55
|
+
- CJK glyphs render correctly (no tofu boxes or missing characters)
|
|
56
|
+
- Reasonable line wrapping
|
|
57
|
+
- Table borders and columns staying readable
|
|
58
|
+
- Page margins and spacing look clean
|
|
59
|
+
- Heading hierarchy visually balanced
|
|
60
|
+
- Long table cells do not overlap adjacent text
|
|
61
|
+
- Row heights expand to fit wrapped content
|
|
62
|
+
|
|
63
|
+
If layout or glyph rendering is wrong, fix font, spacing, or content and re-render.
|
|
64
|
+
|
|
65
|
+
### 5) Cleanup
|
|
66
|
+
|
|
67
|
+
- QA screenshots are temporary — delete after visual check unless the user asks to keep them.
|
|
68
|
+
- Keep only the final PDF as the persistent deliverable.
|
|
@@ -19,10 +19,6 @@ description: Learn and evolve the local skill library from recent Codex conversa
|
|
|
19
19
|
- Quality: Take no action when there are no recent sessions, avoid unrelated broad refactors, keep shared skills cross-project reusable, route project-specific tooling patterns into the relevant project's `~/.codex/skills/`, and validate every changed skill.
|
|
20
20
|
- Output: Report the analyzed sessions, extracted lessons, created or updated skills, shared-vs-project-specific placement decisions, and the reasoning behind each decision.
|
|
21
21
|
|
|
22
|
-
## Overview
|
|
23
|
-
|
|
24
|
-
Extract recent conversations, identify reusable lessons, and convert those lessons into concrete skill updates.
|
|
25
|
-
|
|
26
22
|
## Required Resources
|
|
27
23
|
|
|
28
24
|
- `scripts/extract_recent_conversations.py` for deterministic session extraction, exposed as `apltk extract-skill-conversations`.
|
package/commit-and-push/SKILL.md
CHANGED
|
@@ -19,10 +19,6 @@ description: "Guide the agent to submit local changes with commit and push only
|
|
|
19
19
|
- Quality: Re-run relevant validation for runtime changes, preserve unrelated local work safely when branch switching or post-push local sync is required, do not bypass blocking readiness findings such as missing/stale `Unreleased` bullets or unsynchronized project docs, and never collapse intentionally separated commit scopes just because related unstaged changes are present.
|
|
20
20
|
- Output: Produce a concise Conventional Commit, push it to the intended branch, and report any temporary stash/restore, commit-scope separation, or local branch sync that was required.
|
|
21
21
|
|
|
22
|
-
## Overview
|
|
23
|
-
|
|
24
|
-
Run a standardized commit-and-push workflow without release/version steps.
|
|
25
|
-
|
|
26
22
|
## References
|
|
27
23
|
|
|
28
24
|
Load only when needed:
|
|
@@ -19,10 +19,6 @@ description: "Research specific topics deeply and turn them into evidence-based
|
|
|
19
19
|
- Quality: Distinguish verified facts from analysis or inference and call out limitations, conflicts, or stale data explicitly.
|
|
20
20
|
- Output: Deliver a polished research file with clear sections, citations, dates, and source links.
|
|
21
21
|
|
|
22
|
-
## Overview
|
|
23
|
-
|
|
24
|
-
Use this skill when a user needs topic research that goes beyond a quick answer and should end as a polished file deliverable.
|
|
25
|
-
|
|
26
22
|
## Required Workflow
|
|
27
23
|
|
|
28
24
|
1. Understand the request
|
|
@@ -19,10 +19,6 @@ description: Discover reproducible edge-case risks in changed code or a selected
|
|
|
19
19
|
- Quality: Separate confirmed findings from hypotheses and cover boundary, failure, stateful, and observability edge cases that matter to the scope.
|
|
20
20
|
- Output: Return prioritized findings, edge-case evidence, risk assessment, hardening guidance, and residual risk only.
|
|
21
21
|
|
|
22
|
-
## Overview
|
|
23
|
-
|
|
24
|
-
Use this skill to discover edge-case failures and coverage gaps with evidence-first analysis. The goal is to surface reproducible findings, not to remediate them.
|
|
25
|
-
|
|
26
22
|
## Non-negotiable Boundaries
|
|
27
23
|
|
|
28
24
|
- This skill is discovery-only: do not edit code, do not add or modify tests, and do not open PRs.
|
package/docs-to-voice/SKILL.md
CHANGED
|
@@ -5,13 +5,6 @@ description: Convert text and document content into audio files and sentence-lev
|
|
|
5
5
|
|
|
6
6
|
# Docs to Voice
|
|
7
7
|
|
|
8
|
-
## Dependencies
|
|
9
|
-
|
|
10
|
-
- Required: none.
|
|
11
|
-
- Conditional: none.
|
|
12
|
-
- Optional: none.
|
|
13
|
-
- Fallback: not applicable.
|
|
14
|
-
|
|
15
8
|
## Standards
|
|
16
9
|
|
|
17
10
|
- Evidence: Confirm `project_dir`, input source, mode, and environment-backed settings before generation.
|
|
@@ -19,22 +12,6 @@ description: Convert text and document content into audio files and sentence-lev
|
|
|
19
12
|
- Quality: Respect mode-specific options, sentence splitting rules, and post-process requirements such as `ffmpeg` for speed changes.
|
|
20
13
|
- Output: Return the absolute output audio path together with the generated `.timeline.json` and `.srt` companions.
|
|
21
14
|
|
|
22
|
-
## Overview
|
|
23
|
-
|
|
24
|
-
Use `apltk docs-to-voice` to convert raw text or text files into audio and always save under:
|
|
25
|
-
|
|
26
|
-
`project_dir/audio/{project_name}/`
|
|
27
|
-
|
|
28
|
-
Alongside each audio file, the script also writes:
|
|
29
|
-
|
|
30
|
-
- `{audio_name_without_extension}.timeline.json`
|
|
31
|
-
- `{audio_name_without_extension}.srt`
|
|
32
|
-
|
|
33
|
-
Modes:
|
|
34
|
-
|
|
35
|
-
- `say`: local macOS `say`
|
|
36
|
-
- `api`: Alibaba Cloud Model Studio TTS API (for example `qwen3-tts`)
|
|
37
|
-
|
|
38
15
|
## Workflow
|
|
39
16
|
|
|
40
17
|
1. Collect inputs.
|
|
Binary file
|
|
@@ -28,10 +28,6 @@ description: >-
|
|
|
28
28
|
- Quality: Use `test-case-strategy` to add risk-based tests with property-based, regression, integration, E2E, adversarial, and rollback coverage when relevant.
|
|
29
29
|
- Output: Keep implementation and any planning artifacts traceable, updated, and aligned with actual completion results.
|
|
30
30
|
|
|
31
|
-
## Overview
|
|
32
|
-
|
|
33
|
-
Safely extend brownfield systems by exploring the existing codebase first, using the user's requested change plus discovered impact to decide whether specs are needed, then following a consistent implementation and testing workflow with minimal, well-validated changes.
|
|
34
|
-
|
|
35
31
|
## Workflow
|
|
36
32
|
|
|
37
33
|
### 1) Explore codebase first
|
|
@@ -7,7 +7,7 @@ description: Research the most important tradeable instruments to watch for the
|
|
|
7
7
|
|
|
8
8
|
## Dependencies
|
|
9
9
|
|
|
10
|
-
- Required: `pdf` for the final deliverable.
|
|
10
|
+
- Required: `pdf` for the final deliverable, `cjk-pdf` for CJK font and PDF QA needs.
|
|
11
11
|
- Conditional: none.
|
|
12
12
|
- Optional: none.
|
|
13
13
|
- Fallback: If `pdf` is unavailable, stop and report the missing dependency instead of inventing another export workflow.
|
|
@@ -123,18 +123,7 @@ Do not guess missing facts that materially change the report scope.
|
|
|
123
123
|
- Start from `assets/weekly_market_report_template.md`.
|
|
124
124
|
- Localize headings to the requested output language.
|
|
125
125
|
- Default to Chinese if the user did not specify a language.
|
|
126
|
-
|
|
127
|
-
- On macOS, do not assume a font family is available just because it worked elsewhere.
|
|
128
|
-
- On macOS, prefer a locally verified CJK font in this order when the PDF workflow allows explicit font selection:
|
|
129
|
-
- `/System/Library/Fonts/Hiragino Sans GB.ttc`
|
|
130
|
-
- `/System/Library/Fonts/Supplemental/Songti.ttc`
|
|
131
|
-
- `/Library/Fonts/Arial Unicode.ttf`
|
|
132
|
-
- `/System/Library/Fonts/STHeiti Medium.ttc`
|
|
133
|
-
- `/System/Library/Fonts/STHeiti Light.ttc`
|
|
134
|
-
- Do not hardcode fonts that are missing or known to render poorly on the current macOS host.
|
|
135
|
-
- Do not assume `PingFang` exists on every macOS environment.
|
|
136
|
-
- If the `pdf` skill already has a verified CJK-safe default on the current machine, reuse that default instead of overriding it.
|
|
137
|
-
- Avoid emoji, decorative symbols, and unusual glyphs that often break in PDF rendering.
|
|
126
|
+
Use $cjk-pdf for CJK font selection, content safety, visual QA, and temporary file cleanup.
|
|
138
127
|
- Only include exact price levels, yields, or percentages when they were verified from current sources.
|
|
139
128
|
|
|
140
129
|
### 6) Render the final PDF
|
package/generate-spec/SKILL.md
CHANGED
|
@@ -15,7 +15,7 @@ description: Generate and maintain shared feature planning artifacts (`spec.md`,
|
|
|
15
15
|
## Standards
|
|
16
16
|
|
|
17
17
|
- Evidence: Review the relevant code, configs, and authoritative docs before filling requirements or test plans; when external dependencies, libraries, frameworks, APIs, or platforms are involved, checking their official documentation is mandatory during spec creation.
|
|
18
|
-
- Execution: Generate the planning files from this skill's current templates first
|
|
18
|
+
- Execution: Generate the planning files from this skill's current templates first. Follow the scoping, independence, and parallel-safety rules in Working Rules. Surface shared-file or shared-contract collision risks during planning and resolve coordination rules before implementation starts. Use `preparation.md` only when needed; keep that preparation minimal and free of core business logic or target outcomes (see Working Rules). Complete plans with traceable requirements and risks, handle clarification updates, then wait for explicit approval before implementation.
|
|
19
19
|
- Quality: Keep `spec.md`, `tasks.md`, `checklist.md`, `contract.md`, `design.md`, and any shared batch docs synchronized, use `test-case-strategy` to map each planned test to a concrete risk or requirement, preserve the actual headings and field structure from this skill's templates, and tailor the templates so only applicable items remain active.
|
|
20
20
|
- Output: Store planning artifacts under `docs/plans/{YYYY-MM-DD}/{change_name}/` for single-spec work, or `docs/plans/{YYYY-MM-DD}/{batch_name}/{change_name}/` plus shared `coordination.md` for multi-spec parallel work. Add shared `preparation.md` at the batch root only when prerequisite work must land before member specs can run in parallel, and keep individual specs concise, executable, non-overlapping with that preparation, and easy to update.
|
|
21
21
|
|
|
@@ -46,13 +46,11 @@ Own the shared planning-doc lifecycle for feature work so other skills can reuse
|
|
|
46
46
|
- Keep each spec set scoped to at most three modules.
|
|
47
47
|
- If the requested work would span more than three modules, do not draft one oversized coupled plan.
|
|
48
48
|
- Instead, split the work into multiple spec sets, each independently valid and each covering no more than three modules.
|
|
49
|
-
-
|
|
50
|
-
- For batch generation, treat cross-spec implementation dependency as a planning bug: if Spec B cannot be completed safely until Spec A lands, they should be one spec set or be re-sliced so each spec has its own self-contained outcome.
|
|
49
|
+
- See Working Rules for the independence rule.
|
|
51
50
|
- Use `preparation.md`, not individual spec files, when the only safe way to parallelize a batch is to land shared prerequisite work before any spec starts.
|
|
52
|
-
- Keep `preparation.md`
|
|
53
|
-
- If the required pre-work would change business behavior, implement target logic, or satisfy a meaningful requirement by itself, re-slice it into one or more normal spec sets instead of placing it in `preparation.md`.
|
|
51
|
+
- Keep `preparation.md` small: it may define enabling scaffolds, shared test fixtures, naming/contract stubs, mechanical migrations, or non-business compatibility surfaces. See Working Rules for the preparation.md rule.
|
|
54
52
|
- Allow shared preparation only when it is completed before parallel implementation begins; do not hide required pre-work inside one member spec while other specs depend on it.
|
|
55
|
-
-
|
|
53
|
+
- See Working Rules for the parallel-safety rule.
|
|
56
54
|
- If two candidate spec sets would still need to edit the same non-additive surface, either merge them into one spec set, record a concrete ownership split plus additive-only rule that makes parallel work safe, or move the shared pre-work into `preparation.md` when the pre-work can be completed once before all specs.
|
|
57
55
|
- Use:
|
|
58
56
|
- `WORKSPACE_ROOT=<target_project_root>`
|
|
@@ -121,7 +119,7 @@ Own the shared planning-doc lifecycle for feature work so other skills can reuse
|
|
|
121
119
|
- Create `coordination.md` only when one user request is intentionally split into multiple spec sets that may be implemented in parallel.
|
|
122
120
|
- Place it at `docs/plans/{YYYY-MM-DD}/{batch_name}/coordination.md`.
|
|
123
121
|
- Use it as the batch-level source of truth for shared preparation, ownership boundaries, merge order, and cross-spec constraints.
|
|
124
|
-
-
|
|
122
|
+
- See Working Rules for the independence rule.
|
|
125
123
|
- Record shared fields, shared contracts, or shared data-shape assumptions that multiple spec sets must align on.
|
|
126
124
|
- Record whether the batch is ready for parallel implementation now; if not, point to `preparation.md` for executable prerequisite work or list coordination decisions that must still be settled.
|
|
127
125
|
- When one spec set replaces or removes legacy behavior, state that direction explicitly so all worktrees implement toward the same target rather than preserving the old behavior accidentally.
|
|
@@ -144,10 +142,8 @@ Own the shared planning-doc lifecycle for feature work so other skills can reuse
|
|
|
144
142
|
- Include only the smallest shared prerequisite work that every affected spec can assume after it lands.
|
|
145
143
|
- Exclude core business logic, target business outcomes, user-visible behavior changes, and member-spec implementation guidance; those belong in normal spec files.
|
|
146
144
|
- Include relevant validation work for the preparation itself.
|
|
147
|
-
- Do not include member-spec implementation tasks in `preparation.md`.
|
|
148
145
|
- Remove overlapping instructions from member specs; specs must reference the prepared baseline as an assumption instead of repeating the setup tasks.
|
|
149
|
-
-
|
|
150
|
-
- If the batch can be made parallel-safe through ownership rules, additive-only constraints, or re-slicing, do not create `preparation.md`.
|
|
146
|
+
- See Working Rules for the parallel-safety rule.
|
|
151
147
|
|
|
152
148
|
### 7) Fill `checklist.md`
|
|
153
149
|
|
|
Binary file
|
package/implement-specs/SKILL.md
CHANGED
|
@@ -34,86 +34,38 @@ Implement approved spec planning sets directly in the current checkout when the
|
|
|
34
34
|
|
|
35
35
|
### 1) Identify and read the specs set
|
|
36
36
|
|
|
37
|
-
-
|
|
38
|
-
- If the user provides a specific path, use that directly.
|
|
39
|
-
- If only a `change_name` or date is given, search for matching directories under `docs/plans/`.
|
|
40
|
-
- If the requested path is absent from the current checkout, stop and identify the authoritative source before implementing:
|
|
41
|
-
- inspect the repository history and relevant local branches for that exact `docs/plans/...` path
|
|
42
|
-
- use `recover-missing-plan` when the plan must be restored or reconstructed before work can continue
|
|
43
|
-
- prefer the exact matching plan directory from the authoritative branch or main working tree over archived, approximate, or sibling plan directories
|
|
44
|
-
- if the plan lives under a batch directory, recover only the requested spec directory plus the shared `coordination.md` that governs it
|
|
45
|
-
- do not copy neighboring sibling spec directories into scope unless the user explicitly expands scope
|
|
46
|
-
- When the plan sits under a batch directory, also read the sibling `coordination.md` before implementation.
|
|
47
|
-
- Read all five spec files:
|
|
48
|
-
- `spec.md` — requirements and BDD behaviors
|
|
49
|
-
- `tasks.md` — task breakdown
|
|
50
|
-
- `checklist.md` — behavior-to-test alignment and completion tracking
|
|
51
|
-
- `contract.md` — API/interface contracts
|
|
52
|
-
- `design.md` — design decisions and architecture notes
|
|
53
|
-
- If `coordination.md` exists in the parent batch directory, read it as the shared source of truth for ownership boundaries, shared preparation, replacement direction, merge order, and cross-spec integration checkpoints.
|
|
54
|
-
- Understand the scope, requirements, and planned tasks before proceeding.
|
|
37
|
+
See `../references/implement-specs-common.md` for the standard spec discovery and reading workflow.
|
|
55
38
|
|
|
56
39
|
### 2) Check current branch state
|
|
57
40
|
|
|
58
41
|
- Run `git status -sb` and identify the current branch.
|
|
59
42
|
- Preserve unrelated user changes. If the checkout contains unrelated dirty files, avoid editing them and report any blockers before proceeding.
|
|
60
43
|
- Confirm that the current branch is the intended destination for the implementation. Do not create, rename, or switch branches unless the user explicitly changes scope.
|
|
61
|
-
- If the exact requested plan was recovered into the current checkout, re-read the recovered files before coding so implementation and backfill use the same plan snapshot.
|
|
44
|
+
- If the exact requested plan was recovered (e.g. via `recover-missing-plan`) into the current checkout, re-read the recovered files before coding so implementation and backfill use the same plan snapshot.
|
|
62
45
|
|
|
63
46
|
### 3) Implement the planned tasks
|
|
64
47
|
|
|
65
|
-
-
|
|
66
|
-
- Verify latest authoritative docs for involved stacks/integrations.
|
|
67
|
-
- When `coordination.md` exists, respect its shared-field preparation, legacy-replacement direction, and allowed touch-point boundaries before editing.
|
|
68
|
-
- Implement each task in `tasks.md` systematically.
|
|
69
|
-
- When `coordination.md` defines file ownership guardrails, additive-only shared-contract rules, or compatibility-shim retention requirements, treat them as blocking execution constraints rather than optional guidance.
|
|
70
|
-
- For each implemented change, add appropriate tests:
|
|
71
|
-
- Unit tests for changed logic
|
|
72
|
-
- Regression tests for bug-prone behavior
|
|
73
|
-
- Property-based tests for business logic changes
|
|
74
|
-
- Integration tests for cross-module chains
|
|
75
|
-
- E2E tests for key user-visible paths
|
|
76
|
-
- Adversarial tests for abuse paths
|
|
77
|
-
- Run relevant tests and fix failures.
|
|
78
|
-
- Do not skip testing even for seemingly small changes.
|
|
48
|
+
See `../references/implement-specs-common.md` for the standard implementation workflow.
|
|
79
49
|
|
|
80
50
|
### 4) Backfill completion status
|
|
81
51
|
|
|
82
|
-
|
|
83
|
-
|
|
84
|
-
- Update `spec.md` with actual completion state for each requirement.
|
|
85
|
-
- Mark completed tasks in `tasks.md`.
|
|
86
|
-
- Update `checklist.md` with test execution results, N/A reasons, and any scope adjustments.
|
|
87
|
-
- If the shared implementation direction changed, update the parent `coordination.md` as well before finishing.
|
|
88
|
-
- Do not mark unused template examples or non-applicable items as complete.
|
|
52
|
+
See `../references/implement-specs-common.md` for the standard backfill workflow.
|
|
89
53
|
|
|
90
54
|
### 5) Commit changes
|
|
91
55
|
|
|
92
|
-
-
|
|
93
|
-
```bash
|
|
94
|
-
git add <implementation-files> <test-files> <updated-specs>
|
|
95
|
-
```
|
|
96
|
-
- Write a concise Conventional Commit message describing the implemented scope.
|
|
97
|
-
- Commit to the current branch:
|
|
98
|
-
```bash
|
|
99
|
-
git commit -m "<conventional-commit-message>"
|
|
100
|
-
```
|
|
56
|
+
See `../references/implement-specs-common.md` for the standard commit workflow.
|
|
101
57
|
|
|
102
58
|
### 6) Report completion
|
|
103
59
|
|
|
104
|
-
-
|
|
60
|
+
See `../references/implement-specs-common.md` for the standard reporting format. Add the following context-specific details:
|
|
61
|
+
|
|
105
62
|
- Note the current branch and commit hash.
|
|
106
63
|
- Confirm which tests ran and which planned documents were backfilled.
|
|
107
64
|
|
|
108
65
|
## Working Rules
|
|
109
66
|
|
|
110
67
|
- Always work in the current checkout; never create a branch or git worktree inside this skill unless the user explicitly changes the request.
|
|
111
|
-
-
|
|
112
|
-
- Treat the specs as the source of truth for scope — do not deviate without user approval.
|
|
113
|
-
- When `coordination.md` exists, treat it as the source of truth for batch-level ownership and cutover direction.
|
|
114
|
-
- Never remove a shared shim, rename a shared field, or rewrite a shared file outside the ownership map unless `coordination.md` explicitly allows that change or the user approves a coordination update first.
|
|
115
|
-
- Follow the testing standards from `enhance-existing-features` and `develop-new-features`.
|
|
116
|
-
- Do not push to remote unless the user explicitly requests it.
|
|
68
|
+
- The shared working rules in `../references/implement-specs-common.md` also apply (complete all tasks, treat specs as truth, respect coordination.md, follow testing standards, no remote push unless asked).
|
|
117
69
|
|
|
118
70
|
## References
|
|
119
71
|
|
|
@@ -17,10 +17,9 @@ description: >-
|
|
|
17
17
|
|
|
18
18
|
## Dependencies
|
|
19
19
|
|
|
20
|
-
- Required: `enhance-existing-features
|
|
20
|
+
- Required: `implement-specs`, `enhance-existing-features`, and `develop-new-features` for implementation standards.
|
|
21
21
|
- Conditional: `generate-spec` if spec files need clarification or updates.
|
|
22
|
-
-
|
|
23
|
-
- Fallback: If `enhance-existing-features` or `develop-new-features` is unavailable, stop and report the missing dependency.
|
|
22
|
+
- Fallback: If `implement-specs`, `enhance-existing-features`, or `develop-new-features` is unavailable, stop and report the missing dependency.
|
|
24
23
|
|
|
25
24
|
## Standards
|
|
26
25
|
|
|
@@ -37,24 +36,11 @@ Implement approved spec planning sets in an isolated git worktree, ensuring the
|
|
|
37
36
|
|
|
38
37
|
### 1) Identify and read the specs set
|
|
39
38
|
|
|
40
|
-
-
|
|
41
|
-
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
- prefer the exact matching plan directory from the repository's authoritative branch or main working tree over archived, approximate, or sibling plan directories
|
|
46
|
-
- if the plan lives under a batch directory, sync only the requested spec directory plus the shared `coordination.md` that governs it
|
|
47
|
-
- do not copy neighboring sibling spec directories into the worktree unless the user explicitly expanded scope
|
|
48
|
-
- When the plan sits under a batch directory, also read the sibling `coordination.md` and `preparation.md` before implementation when those files exist.
|
|
49
|
-
- Read all five spec files:
|
|
50
|
-
- `spec.md` — requirements and BDD behaviors
|
|
51
|
-
- `tasks.md` — task breakdown
|
|
52
|
-
- `checklist.md` — behavior-to-test alignment and completion tracking
|
|
53
|
-
- `contract.md` — API/interface contracts
|
|
54
|
-
- `design.md` — design decisions and architecture notes
|
|
55
|
-
- If `coordination.md` exists in the parent batch directory, read it as the shared source of truth for ownership boundaries, replacement direction, merge order, and cross-spec integration checkpoints.
|
|
56
|
-
- If `preparation.md` exists in the parent batch directory, treat it as the already-completed prerequisite baseline for this spec; do not redo its tasks inside the member spec unless the preparation commit is missing or the document says the prerequisite remains blocked.
|
|
57
|
-
- Understand the scope, requirements, and planned tasks before proceeding.
|
|
39
|
+
Use $implement-specs for the standard spec discovery and reading workflow.
|
|
40
|
+
|
|
41
|
+
Additionally:
|
|
42
|
+
|
|
43
|
+
- When `preparation.md` exists in the parent batch directory, treat it as the already-completed prerequisite baseline for this spec; do not redo its tasks inside the member spec unless the preparation commit is missing or the document says the prerequisite remains blocked.
|
|
58
44
|
|
|
59
45
|
### 2) Check current worktree state
|
|
60
46
|
|
|
@@ -104,50 +90,29 @@ Use branch naming from `references/branch-naming.md`.
|
|
|
104
90
|
|
|
105
91
|
### 4) Implement the planned tasks
|
|
106
92
|
|
|
107
|
-
-
|
|
108
|
-
|
|
109
|
-
|
|
93
|
+
Use $implement-specs for the standard implementation workflow.
|
|
94
|
+
|
|
95
|
+
Additionally:
|
|
96
|
+
|
|
110
97
|
- When `preparation.md` exists, implement against its prepared baseline assumptions and avoid duplicating preparation tasks in the member spec.
|
|
111
|
-
- Implement each task in `tasks.md` systematically.
|
|
112
|
-
- When `coordination.md` defines file ownership guardrails, additive-only shared-contract rules, or compatibility-shim retention requirements, treat them as blocking execution constraints rather than optional guidance.
|
|
113
|
-
- For each implemented change, add appropriate tests:
|
|
114
|
-
- Unit tests for changed logic
|
|
115
|
-
- Regression tests for bug-prone behavior
|
|
116
|
-
- Property-based tests for business logic changes
|
|
117
|
-
- Integration tests for cross-module chains
|
|
118
|
-
- E2E tests for key user-visible paths
|
|
119
|
-
- Adversarial tests for abuse paths
|
|
120
|
-
- Run relevant tests and fix failures.
|
|
121
98
|
- When using targeted Rust `cargo test` commands, pass at most one positional test filter per invocation; if multiple selectors are needed, run separate commands or a broader confirmed selector.
|
|
122
|
-
- Treat any targeted test command that executes zero tests as non-verification and rerun with a selector that proves the intended coverage actually ran.
|
|
123
|
-
- Do not skip testing even for seemingly small changes.
|
|
124
99
|
|
|
125
100
|
### 5) Backfill completion status
|
|
126
101
|
|
|
127
|
-
|
|
102
|
+
Use $implement-specs for the standard backfill workflow.
|
|
103
|
+
|
|
104
|
+
Additionally:
|
|
128
105
|
|
|
129
|
-
- Update `spec.md` with actual completion state for each requirement.
|
|
130
|
-
- Mark completed tasks in `tasks.md`.
|
|
131
|
-
- Update `checklist.md` with test execution results, N/A reasons, and any scope adjustments.
|
|
132
|
-
- If the shared implementation direction changed, update the parent `coordination.md` as well before finishing.
|
|
133
106
|
- If preparation assumptions changed or were found missing, update `preparation.md` or stop for re-coordination instead of silently moving prerequisite work into the member spec.
|
|
134
|
-
- Do not mark unused template examples or non-applicable items as complete.
|
|
135
107
|
|
|
136
108
|
### 6) Commit changes
|
|
137
109
|
|
|
138
|
-
-
|
|
139
|
-
```bash
|
|
140
|
-
git add <implementation-files> <test-files> <updated-specs>
|
|
141
|
-
```
|
|
142
|
-
- Write a concise Conventional Commit message describing the implemented scope.
|
|
143
|
-
- Commit to the worktree's local branch:
|
|
144
|
-
```bash
|
|
145
|
-
git commit -m "<conventional-commit-message>"
|
|
146
|
-
```
|
|
110
|
+
Use $implement-specs for the standard commit workflow.
|
|
147
111
|
|
|
148
112
|
### 7) Report completion
|
|
149
113
|
|
|
150
|
-
-
|
|
114
|
+
See $implement-specs for the standard reporting format. Add the following context-specific details:
|
|
115
|
+
|
|
151
116
|
- Note the spec-derived branch name and worktree location.
|
|
152
117
|
- Confirm that the parent branch remains unaffected.
|
|
153
118
|
|
|
@@ -158,14 +123,9 @@ After implementation and testing:
|
|
|
158
123
|
- Treat an already-landed spec as complete work, not as a reason to recreate a duplicate worktree.
|
|
159
124
|
- Keep the new branch based on the same parent branch as the worktree base; do not silently rebase the workflow onto a different branch.
|
|
160
125
|
- Use the spec-set name as the canonical identifier for the branch and worktree unless the user explicitly asks for a different naming scheme.
|
|
161
|
-
- Complete all planned tasks before committing; do not stop with partial work.
|
|
162
|
-
- Treat the specs as the source of truth for scope — do not deviate without user approval.
|
|
163
|
-
- When `coordination.md` exists, treat it as the source of truth for batch-level ownership and cutover direction.
|
|
164
126
|
- When `preparation.md` exists, treat it as a prerequisite baseline owned outside the member spec; do not duplicate or alter its tasks unless explicitly requested.
|
|
165
|
-
- Never remove a shared shim, rename a shared field, or rewrite a shared file outside the ownership map unless `coordination.md` explicitly allows that change or the user approves a coordination update first.
|
|
166
127
|
- Revert formatter-only edits outside the owned spec scope before the final commit so the worktree stays reviewable and merge-safe.
|
|
167
|
-
-
|
|
168
|
-
- Do not push to remote unless the user explicitly requests it.
|
|
128
|
+
- The shared working rules from $implement-specs also apply (complete all tasks, treat specs as truth, respect coordination.md, follow testing standards, no remote push unless asked).
|
|
169
129
|
|
|
170
130
|
## References
|
|
171
131
|
|
|
Binary file
|