Skip to content

Use published sci-brain-slides v0.1.0 package - #65

Merged
GiggleLiu merged 1 commit into
mainfrom
feat/slides-registry-v0.1.0
Sep 18, 2026
Merged

GiggleLiu merged 1 commit into
mainfrom
feat/slides-registry-v0.1.0

Conversation

@GiggleLiu

Copy link
Copy Markdown
Member

Observation

The write-slides skill already targets sci-brain-slides v0.1.0, but its setup still requires cloning the release into a custom Typst package directory.

Issue

Version 0.1.0 is now published in the Typst registry. The manual checkout and package-path instructions are obsolete, and the gallery is not shipped in the registry archive.

Fix

Use typst init @preview/sci-brain-slides:0.1.0 and normal compilation. Link the layout guide, style tokens, and gallery at the upstream v0.1.0 tag. Explain how existing package-based decks can drop their checkout override, and align README.md and CLAUDE.md.

The package version and slide API remain unchanged. First use requires network access; later builds can reuse Typst's cache.

Verification

  • Initialized a fresh deck with empty local-package and cache directories, downloading the published package.
  • Compiled the documented starter command, inline API example, and all five themes at 22 pt without warnings, supplying the required DejaVu Sans font through TYPST_FONT_PATHS.
  • Inspected a rendered starter slide and verified all three version-pinned documentation URLs.
  • python3 scripts/validate_skills.py: 16 skills passed.
  • uv run --with pytest --python /opt/homebrew/bin/python3 pytest -q: 273 passed. Used an isolated environment because the system Anaconda pytest launcher crashes while importing readline.
  • Skill quick validation and git diff --check: passed.

@GiggleLiu

Copy link
Copy Markdown
Member Author

Independent quality review

A fresh-context reviewer examined the complete diff and all three changed files against 18e4cdd.

  • Design: no material findings; registry setup removes checkout management and keeps template ownership upstream.
  • Documentation: README, CLAUDE.md, and skill setup, compilation, and migration instructions agree. Version-pinned upstream links resolve.
  • Reproducibility: imports and initialization retain the explicit 0.1.0 pin.
  • Tests: no tests changed. Author verified 273 passing tests, skill validation, clean-cache package initialization, and warning-free compilation of the example and five themes. Reviewer independently checked whitespace.

Git audit

GATE: PASS

  • harm.secrets: OK. No credentials or sensitive data added.
  • harm.dangerous-files: OK. Only three intended Markdown files changed.
  • harm.generated-artifacts: OK. No generated files committed.
  • harm.cache: OK. No cache directories committed.
  • harm.debug-leftovers: OK. No temporary probes or debug content.
  • pr.description-quality: OK. The PR states the obsolete setup, replacement behavior, and observed verification.
  • pr.traceable: OK. Purpose, validation, and the first-build network requirement are documented.

Critical / Important / Minor findings: none. No corrective action required.

@GiggleLiu
GiggleLiu merged commit ea11c87 into main Sep 18, 2026
1 check 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