Repository navigation
Remove stale links from CS vs GH page - #316
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
marcleblanc2
added a commit
that referenced
this pull request
Oct 27, 2026
…ot alerts Closes the remaining 12 fixable alerts: - mermaid 11 -> 12 brings katex 0.18 and dompurify 3.4.17 within its own declared ranges (#315, #316, #307). Mermaid.tsx pins layout: dagre and look: classic so diagrams render as they did under mermaid 11. - pnpm update --depth 20 brace-expansion source-map-js @grpc/grpc-js dompurify: the parents already allowed the patched versions, the lockfile was stale (#303, #302, #314, #306, #305). - pnpm.overrides for toml (#282, #281) and uuid (#194): their parents under contentlayer2 pin old majors, and contentlayer2 has no newer release. - esbuild as a devDependency (#27): it is only a peer of contentlayer2 and mdx-bundler, which overrides do not reach. Amp-Thread-ID: https://ampcode.com/threads/T-01a17ac5-7379-71a7-997b-fc6a725d4672 Co-authored-by: Amp <amp@ampcode.com>
marcleblanc2
added a commit
that referenced
this pull request
Oct 30, 2026
…efresh) (#2109) ## What Closes 20 of the 24 open [Dependabot alerts](https://github.com/sourcegraph/docs/security/dependabot). - `next` ^16.3.5 → ^16.3.8 (resolves to 16.4.3), `eslint-config-next` to match, `sharp` ^0.35.4 → ^0.35.5 - `mermaid` ^11.16 → ^12.2, which brings `katex` 0.18 and `dompurify` 3.4.17 within its own declared ranges. `Mermaid.tsx` pins `layout: dagre` and `look: classic` so diagrams render as they did under mermaid 11 (12 defaults to ELK layout and the "neo" look). - `pnpm update --depth 20 brace-expansion source-map-js @grpc/grpc-js dompurify`: the parents already allowed the patched versions, the lockfile was just stale. - `pnpm.overrides` for `toml` and `uuid`: their parents under `contentlayer2` pin old majors, and `contentlayer2` has no newer release. `toml` is only used for `+++` TOML frontmatter, which no page has. - `esbuild` ^0.25 as a devDependency: it is only a peer of `contentlayer2` and `mdx-bundler`, which overrides do not reach. | Alert | Severity | Package | Was → now | Fixed by | |---|---|---|---|---| | #304 | critical | next | 16.3.5 → 16.4.3 | bump | | #308, #309, #310, #311, #319, #320 | high … low | next | 16.3.5 → 16.4.3 | bump | | #318 | high | sharp | 0.35.4 → 0.35.5 | bump | | #314 | high | source-map-js | 1.2.1 → 1.2.2 | lockfile refresh | | #306, #305 | high, low | @grpc/grpc-js | 1.14.4 → 1.14.6 | lockfile refresh | | #282, #281 | high | toml | 3.0.0 → 4.3.0 | override | | #303, #302 | medium | brace-expansion | 1.1.18 → 1.1.21, 5.0.9 → 5.0.12 | lockfile refresh | | #194 | medium | uuid | 9.0.1 → 11.1.1 | override | | #27 | medium | esbuild | 0.18.20 → 0.25.12 | devDependency | | #316, #307 | low | dompurify | 3.4.15 → 3.4.17 | mermaid 12 + lockfile refresh | | #315 | low | katex | 0.16.47 → 0.18.10 | mermaid 12 | #304 (RCE in `next/og` `ImageResponse`) is not exploitable here: `src/app/api/og/[...path]/route.tsx` is `force-static` with `dynamicParams = false`, so `ImageResponse` only ever sees page titles from our own MDX at build time. #308 (SSRF in image optimization) does not apply either; `next.config.js` has no `images` config. ## Not in this PR - #234, #205 (`@opentelemetry/propagator-jaeger`, `@opentelemetry/core`): the fix is OTel SDK 2.x, and `contentlayer2` → `@effect-ts/otel` pins 1.x with a different `@opentelemetry/api` peer. That tracing only runs inside `contentlayer2 build` and never reads a request header. Dismiss as "vulnerable code is not actually used". - #317 (`sprintf-js`), #312 (`braces`): withdrawn advisories with no patched version. Dismiss. ## Test plan - `pnpm install`: no new peer warnings (the `lucide-react` / React 19 one is pre-existing); `@opentelemetry/api` stays at 1.9.0 - `npx tsc --noEmit` clean against mermaid 12 - `pnpm run build`: 1494 static pages, OG images render as 1200×630 PNGs - `pnpm run lint`: 0 errors, same 8 pre-existing warnings as `main` - `pnpm start`, loaded `/admin/architecture` and `/self-hosted/sourcegraph-nginx-mermaid` in headless Chrome: diagrams render with the same dagre layout and classic look as mermaid 11 - Vercel preview build on this PR --------- Co-authored-by: Amp <amp@ampcode.com>
This branch was successfully deployed
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.
This PR removes 2 links that had become stale and led to nonexistent pages.
Pull Request approval
Although pull request approval is not enforced for this repository in order to reduce friction, merging without a review will generate a ticket for the docs team to review your changes. So if possible, have your pull request approved before merging.