Skip to content

fix(ci): stop Prettier auto-formatting .trajectories/, exempt it from the proof contract#1653

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
khaliqgant merged 3 commits into from
Sep 4, 2026
+24 −0
Merged

fix(ci): stop Prettier auto-formatting .trajectories/, exempt it from the proof contract #1653

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
c5eb14e
fix(ci): stop Prettier auto-formatting .trajectories/, exempt it from…
Sep 4, 2026
2abc787
ci: also pin npm@11.19.1 in node-compat's fresh-install job
Sep 4, 2026
f9a2503
ci: also pin npm@11.19.1 in package-validation's fresh-install job
Sep 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions .github/workflows/node-compat.yml
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -98,6 +98,14 @@ jobs:
node --version
npm --version

- name: Upgrade npm before fresh resolution
# npm <=10.9.x's arborist crashes re-resolving the harnesses
# package's peer set with no lockfile present (npm/cli#8261) — an
# npm-version bug, not a Node-version one. See relay#1652. Applied
# here too so this job passes independent of merge order with that
# PR.
run: npm install -g npm@11.19.1

- name: Clean install
run: |
rm -rf node_modules package-lock.json
Expand Down
7 changes: 7 additions & 0 deletions .github/workflows/package-validation.yml
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -214,6 +214,13 @@ jobs:
with:
node-version: '22.14.0'

- name: Upgrade npm before fresh resolution
# See node-compat.yml's identical step: npm <=10.9.x's arborist
# crashes re-resolving the harnesses package's peer set with no
# lockfile present (npm/cli#8261). See relay#1652. Applied here too
# so this job passes independent of merge order with that PR.
run: npm install -g npm@11.19.1

# The publish workflow bumps versions, removes package-lock.json, then
# runs npm install. This catches range-resolved dependency breaks before merge.
- name: Install dependencies with publish resolution
Expand Down
1 change: 1 addition & 0 deletions .prettierignore
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -8,3 +8,4 @@ web/content/docs/
packages/config/src/cli-registry.generated.ts
tests/benchmarks/head-to-head.ts
.agentworkforce/trajectories/
.trajectories/
8 changes: 8 additions & 0 deletions scripts/pr-proof/contract.mjs
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -123,6 +123,14 @@ const NON_RUNTIME_PATH_PATTERNS = Object.freeze([
// is the same CI scope already exempted via workflows/ and .github/.
/^\.gitignore$/,
/^\.editorconfig$/,
// Session/trajectory compaction logs (chief-brain journaling for coding
// agent sessions). Never imported by the CLI or broker at runtime — pure
// record-keeping. Nested, so the top-level `*.md` pattern above doesn't
// already cover `.trajectories/compacted/*.md`.
/^\.trajectories\//,
// Formatter config only — cannot change what ships. Same class as
// .editorconfig above.
/^\.prettierignore$/,
]);

/**
Expand Down
Loading