Skip to content

Commit e176da6

Browse files
committed
Merge branch 'fix/security-alerts' into dev
Closes 5 security alerts: - Dependabot #88 (uuid <14) - Dependabot #45 (postcss <8.5.10 transitive via next) - CodeQL #103, #104 (workflow permissions) - CodeQL #108 (vite injector strip) Plus CodeQL paths-ignore for vendored bundles, shadow src, and local audit tools.
2 parents fb906e4 + cb3d21c commit e176da6

7 files changed

Lines changed: 222 additions & 845 deletions

File tree

.github/codeql/codeql-config.yml

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
name: "MLSysBook CodeQL config"
2+
3+
# Paths excluded from analysis. Each entry below is code that operates on
4+
# trusted local artifacts or is vendored from upstream — out of scope for
5+
# the web-facing security analyses CodeQL applies by default.
6+
paths-ignore:
7+
# Vendored third-party widget bundle (minified output from upstream
8+
# socratiQ collaborative-widget-bridge build). Not hand-edited; alerts
9+
# here should be reported upstream, not patched in-tree.
10+
- "book/quarto/tools/scripts/socratiQ/collaborative-widget-bridge.js"
11+
- "book/quarto/tools/scripts/socratiQ/collaborative-widget-bridge.umd.cjs"
12+
13+
# Shadow copy of socratiq client source — parallel to socratiq/js/, kept
14+
# for the shadow-DOM rendering path. Not part of the live web surface.
15+
- "socratiq/src_shadow/**"
16+
17+
# Local audit/maintenance scripts that operate on the user's own Quarto
18+
# build output. Not web-facing; regex-based HTML strip is intentional
19+
# for speed and is safe given trusted input.
20+
- "tools/audit/**"

.github/workflows/staffml-validate-vault.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,10 @@ concurrency:
5151
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
5252
cancel-in-progress: true
5353

54+
# Read-only token: jobs only checkout, install, lint, and test.
55+
permissions:
56+
contents: read
57+
5458
env:
5559
# Workflow-wide single source for paths and versions. See docs/CI-VARIABLES.md.
5660
# The `paths:` trigger filter is intentionally NOT vars-ified — GitHub Actions

0 commit comments

Comments
 (0)