Skip to content

List the build tool and every bundle on the skill page - #18

Merged
alekswebnet merged 2 commits into
mainfrom
feat/skill-page-bundles
Nov 10, 2026
Merged

alekswebnet merged 2 commits into
mainfrom
feat/skill-page-bundles

Conversation

@alekswebnet

@alekswebnet alekswebnet commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Closes #17.

/h5p/skill/ wrote out the full address of each of the h5p-author skill's files, but none of the files its step 3 fetches where npm cannot be reached. A chat with the skill's text alone had to put a bundle's address together from its type, and a fetch tool may refuse that: an outside review of the skill had …/bundles/H5P.Blanks.zip refused so. missing-elements/h5p-offline-player#180 sends such a chat to this page for every address.

What changes

  • The page lists the build tool and every bundle, each at its full address as its link and its text, in a section of its own after the type files. src/_data/skill.js names them by the author package's contentTypes, the list its writeBundles writes from, so the page, written before Vite writes the bundles, keeps no copy of the rule. Both are public exports of h5p-author 0.9.0: this needs no release and can merge first.
  • The SKILL.md test takes either release's addresses. It no longer expects /h5p/build/types/<Type>.md, which 0.9.0 names and the next release does not; it still fetches every address SKILL.md gives. So the pin can move without changing the test.
  • The comment in vite.config.ts says the page names the bundles.

Tested

  • pnpm build, pnpm typecheck, pnpm e2e: 16 pass. The new test holds the page's bundle links to /h5p/bundles/index.json, which Vite wrote: a page missing a bundle, or writing an address that is not whole, fails it (both tried).
  • With the next release's SKILL.md served in its place, the SKILL.md test passes.
  • On Haiku, in a stand-in for a chat whose fetch tool opens only addresses seen whole, with npm cut off and this build of the site, given the next release's SKILL.md: all six runs opened this page and took every address from it, nothing refused, and the two given the build tool as served that kept to the bundle built a package that plays (details in h5p-author skill: a chat with the skill's text alone is given addresses to put together, which a fetch tool may refuse h5p-offline-player#180's pull request).

Summary by cubic

The skill page now lists the build tool and every bundle it serves at their full addresses, so a chat that cannot reach npm can fetch the files step 3 of SKILL.md names without assembling bundle addresses from type names.

  • The page lists h5p-author.mjs and every bundle after the type files, each at its full address as its link and its text, named from the author package's contentTypes — the same list writeBundles writes from — so the page keeps no copy of the rule and this needs no release.
  • The SKILL.md test accepts either release's addresses: it no longer expects /h5p/build/types/<Type>.md, which h5p-author 0.9.0 names and the next release does not, and it holds the skill to a way to a type's file — 0.9.0's address or this page, which writes each one out — so the pin can move without changing the test.

Closes #17.

Written for commit 8c7bd4c. Summary will update on new commits.

View guided diff Turn on auto-fix

/h5p/skill/ wrote out the full address of each of the skill's files but
none of the files its step 3 fetches where npm cannot be reached, so a
chat with the skill's text alone had to put a bundle's address together
from its type, which a fetch tool may refuse. The page now lists
h5p-author.mjs and every bundle /h5p/bundles/ serves, each at its full
address as its link and its text, named by the author package's
contentTypes, the list writeBundles writes from. The SKILL.md test no
longer expects the type-file address 0.9.0 names, so the pin can move
to the next release without changing it.

Closes #17

@cubic-dev-ai cubic-dev-ai 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.

All reported issues were addressed across 4 files

Reply with feedback, questions, or to request a fix.

View guided diff | Turn on auto-fix | Re-trigger cubic

Comment thread e2e/skill.spec.ts Resolved
The SKILL.md test stopped expecting 0.9.0's type-file address so the
pin could move, and with it any check that the skill gives a way to a
type's file at all (cubic's review). It now asks for 0.9.0's address of
its own or this page, which writes each one out, as the next release
names it. SKILL.md with neither fails it.
@alekswebnet
alekswebnet merged commit 1679a5b into main Nov 10, 2026
4 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.

/h5p/skill/ lists no bundle: a chat building where npm cannot be reached puts their addresses together

1 participant