Skip to content

[codex] Tighten savepoint skill docs#26

Merged
dd3ok merged 4 commits into
mainfrom
codex/savepoint-skill-lite
Jun 13, 2026
Merged

[codex] Tighten savepoint skill docs#26
dd3ok merged 4 commits into
mainfrom
codex/savepoint-skill-lite

Conversation

@dd3ok

@dd3ok dd3ok commented Jun 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • Tighten skills/savepoint/SKILL.md to a compact contract while preserving save/load/text modes and recovery guardrails.
  • Add repo validation checks that keep the skill concise and keep direct CLI shortcut detail out of the skill body.
  • Slim English and Korean READMEs by removing old migration/key mapping tables and long maintainer validation blocks, replacing them with focused maintainer/reference links.
  • Add the README boundary that Savepoint is a recoverable repo/Git checkpoint, not a lightweight conversation summary.

Why

The feedback is valid for this repository: Savepoint should not be reduced to a tiny handoff summary, but the skill should also avoid carrying README-level or maintainer-level detail. The actual repo reference names are references/contract.md, references/safety.md, and references/template.md, so the skill keeps those names instead of the longer names in the pasted draft.

Validation

  • python3 scripts/check-frontmatter.py
  • python3 scripts/check-marker-block.py
  • python3 scripts/check-marker-semantics.py
  • python3 scripts/validate-examples.py
  • python3 scripts/check-output-contract.py
  • python3 scripts/validate-repo.py
  • python3 scripts/check-savepoint-renderer.py (Windows symlink privilege case skipped; remaining tests ok)
  • python3 scripts/check-install-helper.py (Windows symlink privilege case skipped; remaining tests ok)
  • python3 scripts/savepoint.py validate --allow-example-paths examples/SAVEPOINT.filled.example.md examples/file-bugfix/SAVEPOINT.md examples/file-architecture/SAVEPOINT.md examples/unsafe-savepoint/SAVEPOINT.md
  • git diff --check
  • git diff --cached --check
  • python3 -m compileall -q skills/savepoint/scripts scripts

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request simplifies and streamlines the documentation in README.md, README.ko.md, and SKILL.md by removing detailed maintainer/migration commands, CLI shortcut details, and multi-line code blocks. It also updates scripts/validate-repo.py to enforce these new constraints, including a strict line count limit of 55–65 lines for SKILL.md. A review comment correctly points out that SKILL.md currently has 66 lines, which would fail this new validation check, and suggests removing an empty line to meet the requirement.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread skills/savepoint/SKILL.md Outdated
Comment on lines 6 to 7

# Savepoint

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The current line count of SKILL.md is 66 lines, which will fail the repository validation check in scripts/validate-repo.py (which strictly enforces a limit of 55-65 lines). Removing the unnecessary empty line right after the frontmatter delimiter reduces the total line count to exactly 65 lines, satisfying the validation contract.

Suggested change
# Savepoint
# Savepoint

@dd3ok dd3ok marked this pull request as ready for review June 13, 2026 03:49
@dd3ok dd3ok merged commit 9c7f1ec into main Jun 13, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant