fix: clean broken doc references, add shrink test script, pin CI actions, and skip Go tests gracefully - #826
Open
poorvith-mp wants to merge 1 commit into
Conversation
…ons, and skip Go tests gracefully - Replace unresolvable relative markdown links pointing to internal monorepo specs and omitted paths with plain text across explorer, skills, packages, engine, and proxy docs. - Add "test" script and package smoke test to shrink/package.json and shrink/tests/package.test.mjs. Fix prepack path in shrink/package.json. - Pin third-party Actions in .github/workflows/ci.yml to full commit SHAs with version comments (matching engine-ci.yml). - Add Go toolchain existence checks to packages/cli runtime test suites so Go-dependent tests skip gracefully instead of failing when Go is absent. - Add CAVEMAN_DEBUG diagnostic logging for silent fs.unlinkSync catches in hooks and opencode plugin. - Fix cwd and permission handling in standalone hook and symlink test suites on Windows. Closes JuliusBrussee#825 Signed-off-by: Poorvith M P <poorvith007@proton.me>
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.
Summary
This PR addresses several gaps and broken references identified in repo;
docs/decisions/*,docs/prd/*,docs/CAVEMAN_LEARN_SPEC.md,docs/FASTCONTEXT_EXPLORER_SPEC.md,docs/research/*,PRODUCTS.yaml,examples/coding-agent) with plain text code references across package docs.shrinktest script: Adds"test": "node --test tests/*.test.mjs"and package smoke testing inshrink/, fixing theprepackpath from../clito../packages/cli.actions/checkout,actions/setup-node,actions/setup-python) in.github/workflows/ci.ymlto full commit SHAs with version comments (matchingengine-ci.yml).execFileSync("go", ...)inpackages/cliruntime test suites with a toolchain existence check so tests skip cleanly (t.skip("go toolchain not found")) in Node-only environments without Go on PATH.if (process.env.CAVEMAN_DEBUG === '1')logging for silentfs.unlinkSynccatches in hooks and opencode plugin.Closes #825
Verification
npm test: 152 tests (145 passed, 7 skipped, 0 failed).node --test packages/cli/tests/agent-compression-conformance.runtime.mjs packages/cli/tests/harness-agent-upstream.runtime.mjs packages/cli/tests/harness-session-history.runtime.mjs packages/cli/tests/wrap-restart.runtime.mjs: 10 tests skipped cleanly when Go is absent.node --test shrink/tests/package.test.mjs: 1 passed.node --test mcp/tests/package.test.mjs: 1 passed.node tests/test_*.js: all standalone hook suites passed.Signed-off-by: Poorvith M P poorvith007@proton.me