Skip to content

Harden packaging & smoke tests; clarify adapter compatibility wording#26

Merged
HiveForensicsAI merged 1 commit intocodex/restructure-knolo-repository-for-monorepofrom
codex/fix-adoption-blockers-for-create-knolo-app
Feb 21, 2026
Merged

Harden packaging & smoke tests; clarify adapter compatibility wording#26
HiveForensicsAI merged 1 commit intocodex/restructure-knolo-repository-for-monorepofrom
codex/fix-adoption-blockers-for-create-knolo-app

Conversation

@HiveForensicsAI
Copy link
Contributor

Motivation

  • Ensure @knolo/cli is publish-ready with a correct bin entry, included runtime files, and a runtime semver dependency on @knolo/core so generated apps install outside the monorepo.
  • Make the create-knolo-app template and its smoke tests validate real-world install/build behavior using packed tarballs instead of workspace-only references.
  • Remove ambiguous language that could be interpreted as claiming official framework integrations for the adapters and make adapter positioning consistent across packages and examples.

Description

  • Hardened packages/cli/package.json for npm publishing by setting private: false, a real version (0.3.1), canonical bin mapping ("knolo": "bin/knolo.mjs"), files: ["bin/"], and a runtime dependency on @knolo/core: "^0.3.1".
  • Added a CLI tarball test (packages/cli/test/cli.test.mjs) that runs npm pack, inspects the tarball entries, asserts package/bin/knolo.mjs and package/package.json exist, and verifies test/ and src/ are not included and package.json fields are correct.
  • Strengthened create-app template and smoke test (packages/create-knolo-app/template/package.json and packages/create-knolo-app/test/create-knolo-app.test.mjs) to use npm pack for @knolo/core, @knolo/cli, and create-knolo-app, scaffold via the packed create-app package, replace deps with local tarball paths, run npm install and npm run knolo:build, and assert dist/knowledge.knolo is produced.
  • Tightened README and examples to use “LangChain-style” / “LlamaIndex-style” wording and added a clear compatibility note that the adapter packages provide interface-compatible shapes inspired by those frameworks and do not depend on or subclass the official framework classes; updated adapter package descriptions and test titles for consistent positioning.

Testing

  • From the repo root I ran npm install, npm run build, and npm test, and the full workspace test suite completed successfully with all tests passing.
  • The new CLI tarball test executed npm pack on packages/cli, inspected the produced tarball, and passed assertions that bin/knolo.mjs is present and package.json metadata (private: false, bin mapping, @knolo/core: ^0.3.1) are correct.
  • The strengthened create-app smoke test packed @knolo/core, @knolo/cli, and create-knolo-app, scaffolded a project from the packed create-app package, installed the packed tarball dependencies, ran npm run knolo:build, and asserted that dist/knowledge.knolo exists (test passed).

Codex Task

@HiveForensicsAI HiveForensicsAI merged commit bead993 into codex/restructure-knolo-repository-for-monorepo Feb 21, 2026
0 of 2 checks passed
@HiveForensicsAI HiveForensicsAI deleted the codex/fix-adoption-blockers-for-create-knolo-app branch February 21, 2026 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant