fix(release): ship built-in skills in the hypha-skills package - #22
Merged
Conversation
The framework built-in skills (context-enrichment, intent-classification) live only under apps/server/src/core/skills/builtins, which is not part of any published npm package. Consumers of the @codesoul-co/hypha-* release line therefore cannot load the built-ins without a Server source checkout. Move the canonical files into packages/skills/builtins, include the directory in the published tarball, and export resolveBuiltinSkillsDirectory() so consumers and the Server share one source of truth. The Server's SkillManager now resolves its default builtin directory from the package instead of process.cwd(). Adds a package-level test proving both built-ins load from the packaged directory.
erwinmsmith
added a commit
that referenced
this pull request
Aug 14, 2026
The published 1.0.0 line predates two fixes now on main: - #22 ships the framework built-in skills inside @codesoul-co/hypha-skills (tarball data + resolveBuiltinSkillsDirectory export); - #23 resolves the npm audit findings that fail the Repository gates job. The patch release moves the whole release line to 1.0.1 with exact-version internal dependencies.
erwinmsmith
added a commit
to erwinmsmith/THETA-Agent
that referenced
this pull request
Aug 14, 2026
Hypha is no longer a local third_party checkout. THETA-Agent now depends on the published @codesoul-co/hypha-* npm release line (1.0.1) exactly as Hypha's official release guidance prescribes; THETA remains the single third_party upstream checkout. - manifests: replace all file:third_party/Hypha dependencies with @codesoul-co/hypha-*@1.0.1 and drop the hypha:install/hypha:build steps - imports: rename every @hypha/* import across agent, tools, domain to the published package names - skills: load Hypha built-in skills from the shipped resolveBuiltinSkillsDirectory() data of @codesoul-co/hypha-skills instead of a source checkout - doctor: replace hypha.lock/hypha.build checks with hypha.packages, which verifies the installed release line, package entries, and shipped built-ins - upstreams: config/upstreams.lock.json now pins only THETA - tooling: allow better-sqlite3 postinstall for pnpm; update repository validation, README, architecture and CLI docs accordingly Hypha-side fixes released upstream and consumed here: - CodeSoul-co/Hypha#22 ships built-in skills in the hypha-skills tarball - CodeSoul-co/Hypha#23 resolves the npm audit failures - CodeSoul-co/Hypha#24 prepares and publishes @codesoul-co/hypha-* 1.0.1 - CodeSoul-co/Hypha#25 makes the official consumer example compile from the registry Verified: build, test:registries, test:providers, test:docs, test:python, validate, doctor, governed demo, REPL, and the HTTP API all pass without a Hypha checkout.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The framework built-in skills (
context-enrichment,intent-classification) live only underapps/server/src/core/skills/builtins. The published@codesoul-co/hypha-*release line does not include the Server, so npm consumers (e.g. agents that compile a DomainPack referencing the built-ins) cannot load them without a source checkout.Change
packages/skills/builtins/.builtins/in the@codesoul-co/hypha-skillstarball (files: ["dist", "builtins"]).resolveBuiltinSkillsDirectory()from@codesoul-co/hypha-skillsso consumers and the Server share one source of truth.SkillManagerresolves its default builtin directory from the package instead ofprocess.cwd().packages/skills/src/builtin-skills.test.tsproving both built-ins load from the packaged directory.docs/guides/releases.md.Verification
npm run build:packages✅npx vitest run packages/skills/src/builtin-skills.test.ts packages/skills/src/skills.test.ts(14 tests) ✅npx tsc -p apps/server/tsconfig.json --noEmit✅npm run release:check:npm✅ (15 publishable packages at 1.0.0)