fix(release): correct pnpm setup order and align with ci.yml pattern#71
Merged
Conversation
In both the test and publish jobs, pnpm/action-setup@v4 now runs before actions/setup-node so that pnpm is available when setup-node configures the pnpm cache. Removes the corepack steps that were racing ahead of pnpm installation, and pins pnpm to version 9 (matching ci.yml). Adds the missing cache: pnpm option to the publish job. Node 20 is now the only version used across all workflows. https://claude.ai/code/session_01AvPNNJF63SxWKCz93wW2e5
…ating tests Adds workflow_call trigger to ci.yml so it can be invoked by other workflows. The release test job now delegates to ci.yml rather than maintaining a duplicate copy of the same steps, so tests run identically whether triggered by a push/PR or a tag release. https://claude.ai/code/session_01AvPNNJF63SxWKCz93wW2e5
- Bump Node.js requirement 18 → 20 throughout - Add Operational settings table (GITSEMA_VERBOSE, GITSEMA_REMOTE, GITSEMA_LLM_URL, GITSEMA_LOG_MAX_BYTES) - Add GITSEMA_MAX_BODY_SIZE to HTTP server rate-limiting table - Add embedeer as a third provider option - Expand index start flags: --include-glob, --embed-batch-size, --remote, --text-model, --code-model, --quantize, --build-vss, --auto-build-vss, --level, --profile - Expand search flags: --level, --threshold, --narrate, --out, --vss, --repos, --model - Expand first-seen flags: --hybrid, --bm25-weight, --include-commits, --narrate, --out - Update command groups table with all current command groups - Update goal-based quick-finder with new commands and --out/--narrate - Add gitsema quickstart section - Add gitsema repl section - Add gitsema code-review section - Add gitsema workflow run/list section with all 8 templates - Add gitsema triage section - Add gitsema policy section - Add gitsema ownership section - Add --out unified output documentation (all output formats) - Add --narrate LLM summary documentation - Fix strategic review link: review4.md → review7.md https://claude.ai/code/session_01AvPNNJF63SxWKCz93wW2e5
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
2 participants
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.
In both the test and publish jobs, pnpm/action-setup@v4 now runs before
actions/setup-node so that pnpm is available when setup-node configures
the pnpm cache. Removes the corepack steps that were racing ahead of
pnpm installation, and pins pnpm to version 9 (matching ci.yml). Adds
the missing cache: pnpm option to the publish job. Node 20 is now the
only version used across all workflows.
https://claude.ai/code/session_01AvPNNJF63SxWKCz93wW2e5