fix(ci): minimal bootstrap repair #22192
There are no files selected for viewing
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -21,7 +21,7 @@ jobs: | |
| version: 9.15.4 | ||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 24 | ||
| cache: pnpm | ||
| - run: pnpm install --frozen-lockfile | ||
| - run: pnpm exec tsx .github/scripts/validate-evidence.ts | ||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -18,7 +18,7 @@ jobs: | |
| - uses: actions/checkout@v4 | ||
| - run: mkdir -p metrics | ||
| - run: echo '{"pr":0,"ttm_ms":0,"version":"1.0.0"}' > metrics/merge_latency.json | ||
| - run: pnpm install --frozen-lockfile || true | ||
|
There was a problem hiding this comment. Missing pnpm setup - command will silently fail.
🔧 Proposed fix: Add pnpm setup schemas:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
+ - uses: pnpm/action-setup@v4
- run: mkdir -p metrics
- run: echo '{"pr":0,"ttm_ms":0,"version":"1.0.0"}' > metrics/merge_latency.json
- - run: pnpm install --frozen-lockfile || true
+ - run: pnpm install --frozen-lockfile
- run: node .repoos/scripts/ci/validate_schemas.mjs🤖 Prompt for AI Agents |
||
| - run: node .repoos/scripts/ci/validate_schemas.mjs | ||
|
|
||
| checksum: | ||
|
|
||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -34,7 +34,7 @@ jobs: | |
| - name: Setup Node | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 24 | ||
|
|
||
| - name: Install Dependencies | ||
| run: npm i -g pnpm && pnpm install --frozen-lockfile | ||
|
|
||
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
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -351,7 +351,7 @@ jobs: | |||||
| - name: Verify Workspace Boundary | ||||||
| run: node scripts/ci/verify_workspace_boundary.mjs | ||||||
| - name: Verify governance docs | ||||||
| run: ppnpm install --frozen-lockfile:docs-governance | ||||||
|
There was a problem hiding this comment. Typo: Same typo as in other workflow files. 🐛 Proposed fix- run: ppnpm install --frozen-lockfile:docs-governance
+ run: pnpm ci:docs-governance📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||
|
|
||||||
| soc-controls: | ||||||
| name: SOC Controls (ci-pr) | ||||||
|
|
||||||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -76,7 +76,7 @@ jobs: | |
| - name: Install Node.js | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 24 | ||
|
|
||
| - name: Install Dependencies | ||
| run: pnpm install --frozen-lockfile | ||
|
|
||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -29,7 +29,7 @@ jobs: | |
| <<<<<<< HEAD | ||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 24 | ||
| cache: pnpm | ||
| - run: pnpm install --frozen-lockfile | ||
| - run: node .github/scripts/verify-regulatory-ew-evidence.ts | ||
|
|
||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -57,7 +57,7 @@ | |
|
|
||
| - name: Restore npm cache (if enabled) | ||
| if: inputs.cache-enabled | ||
| uses: actions/cache@v4 | ||
| with: | ||
| path: ~/.npm | ||
| key: ${{ runner.os }}-npm-${{ hashFiles('**/package-lock.json') }} | ||
|
|
@@ -65,7 +65,7 @@ | |
| ${{ runner.os }}-npm- | ||
|
|
||
| - name: Install dependencies | ||
| run: pnpm install --frozen-lockfile --prefer-offline --no-audit | ||
|
|
||
| - name: Run linting | ||
| run: npm run lint --if-present | ||
|
|
||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -86,7 +86,7 @@ jobs: | |
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 24 | ||
| - run: pnpm install --frozen-lockfile | ||
| - run: node scripts/datasets/validate-datasets.mjs | ||
|
Comment on lines
86
to
90
There was a problem hiding this comment. 🧩 Analysis chain🏁 Script executed: #!/bin/bash
set -euo pipefail
rg -n -C3 'dataset-validation|pnpm/action-setup|pnpm install --frozen-lockfile' .github/workflows/dataset-flywheel.ymlRepository: BrianCLong/summit Length of output: 1225 Add Line 89 calls Add the following step after the checkout action and before setup-node: - uses: pnpm/action-setup@v4
with:
version: 9.15.4🤖 Prompt for AI Agents |
||
| - name: duplicate task-id check | ||
| run: node scripts/datasets/validate-no-duplicate-task-ids.mjs | ||
|
|
||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -25,7 +25,7 @@ jobs: | |
| python-version: '3.11' | ||
|
|
||
| - name: Install deps | ||
| run: echo "Skip missing requirements.txt" | ||
|
Comment on lines
27
to
+28
There was a problem hiding this comment. 🧩 Analysis chain🏁 Script executed: #!/bin/bash
# Check if requirements.txt exists
fd -t f 'requirements.txt' ci/
# Check what dependencies the script actually needs
fd -t f 'check_embedding_drift.py' --exec head -50 {}
# Look for import statements in the drift check script
rg -n '^import |^from ' ci/ --glob '*.py'Repository: BrianCLong/summit Length of output: 14055 🏁 Script executed: cat .github/workflows/embedding-drift-gate.ymlRepository: BrianCLong/summit Length of output: 2397 Skipping dependency installation will break downstream Python execution. The Either:
🤖 Prompt for AI Agents |
||
|
|
||
| - name: Enforce emit-only policy for provenance updates | ||
| if: github.event_name == 'pull_request' | ||
|
|
||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -18,7 +18,7 @@ jobs: | |
| - uses: pnpm/action-setup@v4 | ||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 24 | ||
| cache: pnpm | ||
| - run: pnpm install --frozen-lockfile | ||
| - run: pnpm --filter intelgraph-server build | ||
|
|
@@ -31,7 +31,7 @@ jobs: | |
| - uses: pnpm/action-setup@v4 | ||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 24 | ||
| cache: pnpm | ||
| - run: pnpm install --frozen-lockfile | ||
| - run: pnpm --filter intelgraph-server test -- demo.seed.test.ts demo.reset.test.ts | ||
|
|
@@ -44,7 +44,7 @@ jobs: | |
| - uses: pnpm/action-setup@v4 | ||
| - uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 24 | ||
| cache: pnpm | ||
| - run: pnpm install --frozen-lockfile | ||
| - run: pnpm --filter intelgraph-server test -- ga.demo.smoke.test.ts | ||
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
| Original file line number | Diff line number | Diff line change | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -41,7 +41,7 @@ jobs: | |||||||||||
| pnpm -v | ||||||||||||
| pnpm install --frozen-lockfile | ||||||||||||
| elif [ -f package-lock.json ]; then | ||||||||||||
| pnpm install --frozen-lockfile | ||||||||||||
|
Comment on lines
43
to
+44
There was a problem hiding this comment. 🧩 Analysis chain🏁 Script executed: #!/bin/bash
set -euo pipefail
FILE=".github/workflows/ga_blocker_radar.yml"
echo "== Install deps block =="
awk '
/- name: Install deps \(best effort\)/ {in_block=1}
in_block {print}
in_block && /^[[:space:]]*-[[:space:]]name:/ && $0 !~ /Install deps \(best effort\)/ {exit}
' "$FILE"
echo
echo "== Relevant commands =="
rg -n 'package-lock\.json|pnpm install --frozen-lockfile|npm ci|corepack enable' "$FILE"
echo
echo "== Lockfiles present in repo (sample) =="
fd -t f 'package-lock\.json$' | head -n 20 || true
fd -t f 'pnpm-lock\.yaml$' | head -n 20 || trueRepository: BrianCLong/summit Length of output: 1644 Fix At Line 44, the Proposed fix elif [ -f package-lock.json ]; then
- pnpm install --frozen-lockfile
+ npm ci📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||
| elif [ -f yarn.lock ]; then | ||||||||||||
| corepack enable | ||||||||||||
| yarn install --frozen-lockfile | ||||||||||||
|
|
||||||||||||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -45,7 +45,7 @@ jobs: | |
| cache: 'npm' | ||
|
|
||
| - name: Install deps (no scripts) | ||
| run: pnpm install --frozen-lockfile --ignore-scripts | ||
|
|
||
| - name: Build | ||
| run: npm run build --if-present | ||
|
|
||
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Critical typo:
ppnpmis not a valid command.This line contains two errors:
ppnpmis a typo — should bepnpm--frozen-lockfile:prod-guardis invalid syntax — colons don't work this way in pnpm flagsBased on the AI summary indicating the original command was
pnpm ci:prod-guard, this appears to be a malformed edit.🐛 Proposed fix
📝 Committable suggestion
🤖 Prompt for AI Agents