Skip to content

docs(registry): clarify catalog preview asset path - #1949

Closed
tianma-if wants to merge 1 commit into
heygen-com:mainfrom
tianma-if:codex/registry-preview-path-doc
Closed

docs(registry): clarify catalog preview asset path#1949
tianma-if wants to merge 1 commit into
heygen-com:mainfrom
tianma-if:codex/registry-preview-path-doc

Conversation

@tianma-if

Copy link
Copy Markdown
Contributor

Summary

  • Clarify that catalog preview cards use uploaded static asset URLs.
  • Include the generated skills manifest hash update from the pre-commit hook.

Validation

  • git diff --check -- skills/hyperframes-registry/references/contributing.md

@tianma-if
tianma-if marked this pull request as ready for review July 5, 2026 08:08
@tianma-if

Copy link
Copy Markdown
Contributor Author

Ready for review. This PR is mergeable, kept to a small single-purpose change, and maintainer edits are enabled. Happy to adjust anything you prefer before merge.

@miguel-heygen

Copy link
Copy Markdown
Collaborator

@tianma-if why the skills manifest was modified?

@miguel-heygen

Copy link
Copy Markdown
Collaborator

Closing in the backlog value sweep. This is documentation, test-only, dependency upkeep, refactoring, or narrow diagnostic/ergonomic polish without enough current user impact or an active merge path to justify carrying it open. Please reopen or resubmit against current main if the need is still concrete.

@tianma-if

Copy link
Copy Markdown
Contributor Author

The manifest changed because skills/hyperframes-registry/references/contributing.md is part of the published skill content. The repository pre-commit hook regenerates skills-manifest.json, and CI checks that its per-skill hash matches skills/. I refreshed this PR onto current main and regenerated the manifest with bun packages/cli/scripts/gen-skills-manifest.ts; the manifest check and formatting now pass.

@jrusso1020

Copy link
Copy Markdown
Collaborator

Thanks for the contribution! The current catalog preview staging and upload workflow is now documented in #3676, with your authorship preserved. Closing this older PR as superseded.

@jrusso1020 jrusso1020 closed this Sep 4, 2026
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.

3 participants