docs: troubleshooting, FORCE, Nx run, and CI ignore-scripts - #33
Merged
simoncrypta merged 3 commits intoAug 10, 2026
Conversation
📝 WalkthroughSummary by CodeRabbit
WalkthroughChangesThe documentation updates replace the Nx acquisition pattern with Documentation workflows
Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@README.md`:
- Line 443: Update the README cleanup guidance and the automatic cold-start
pruning to scope deletion to resources owned by the current CI job, using a
job-specific directory or equivalent identifier for all /dev/shm paths. If that
cannot be enforced, explicitly state that the cleanup is only safe on an
isolated runner; apply the same safety scope to both manual commands and
automatic pruning.
- Around line 317-328: Update the “Yarn Berry / ignore-scripts consumers”
example to account for Plug’n’Play, either by explicitly limiting it to projects
using nodeLinker: node-modules or by resolving the `@cedarjs/pg` package path
through Yarn before invoking ci-install-autopg.sh.
- Around line 348-357: Update the README Jest lifecycle documentation around the
setupFilesAfterEnv configuration and troubleshooting entry to state that both
setupFiles and setupFilesAfterEnv run once per test file, including files
sharing a worker. Correct the cloneWorkerDatabase guidance to explain that its
module-level memo is not process-wide and the example may create one clone per
test file rather than one per worker.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 2ef0c855-619e-4278-b076-795426ba2d5a
📒 Files selected for processing (3)
CHANGELOG.mdREADME.mdsrc/adapters/nx.ts
simoncrypta
force-pushed
the
08-09-docs_troubleshooting_force_nx_run_and_ci_ignore-scripts
branch
from
August 10, 2026 02:43
52d99f5 to
6ff2dcf
Compare
simoncrypta
deleted the
08-09-docs_troubleshooting_force_nx_run_and_ci_ignore-scripts
branch
August 10, 2026 02:46
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
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
db:ready(createAcquireTask+force) +cedarpg run --force.CEDAR_PG_FORCE=1once injest.config.js; prefersetupFilesAfterEnv.YARN_ENABLE_SCRIPTS=falseCI copy-paste forci-install-autopg.sh./dev/shmsizing + leftover prune notes; new Troubleshooting table.0.1.0→ stable1.0.0/ current0.2.0-alpha.x).Depends on #32 (stacked on #31).
Test plan
vp checkvp test tests/nx.test.ts