Skip to content

[CD-1226] Remediate starter app package vulnerabilities#35

Open
andrazk wants to merge 1 commit into
mainfrom
codex/cd-1226-remediate-starter-vulns
Open

[CD-1226] Remediate starter app package vulnerabilities#35
andrazk wants to merge 1 commit into
mainfrom
codex/cd-1226-remediate-starter-vulns

Conversation

@andrazk

@andrazk andrazk commented Jun 16, 2026

Copy link
Copy Markdown

Summary

Remediates the GitHub Dependabot medium Next.js findings for CaseMark/casedotdev-starter-app by updating:

  • next to 16.2.6
  • eslint-config-next to 16.2.6

Vanta

https://app.vanta.com/c/casemark/tests/packages-checked-for-vulnerabilities-v2-records-closed-github-dependabot-medium

Validation

  • bun install --frozen-lockfile
  • Targeted bun audit --json check for next returned {}
  • bun run build passed

Note: full bun audit still reports unrelated advisories outside this PR scope.

@thurgood-bot

thurgood-bot Bot commented Jun 16, 2026

Copy link
Copy Markdown

DB Cooper Database Review

No database-related code, schema, or migration changes detected in this PR; dependency bumps only. No DB risk identified.

No database risks found. Migrations and DB-impacting changes look safe.


DB review by DB Cooper | Re-run with /dbreview

@thurgood-bot

thurgood-bot Bot commented Jun 16, 2026

Copy link
Copy Markdown

Oppenheimer Cleanup Review

No production code changes in this PR; it is a dependency upgrade only. The only potential simplification is lockfile pruning, but it cannot be proven safe without validation of Bun’s resolution rules.

⚠️ Risk If Removed (1)

Removing these items could have subtle side effects. Review the documented risks before proceeding.

  • bun.lock:1361: The lockfile now includes an extra alias entry browserslist/baseline-browser-mapping alongside the top-level baseline-browser-mapping. Repo-wide search shows only these lockfile entries; no source references. This may be a Bun lockfile artifact, but removing it could change dependency resolution or integrity.

    Keep as-is unless you can regenerate bun.lock with the same Bun version and confirm the alias entry is unnecessary via bun install --frozen-lockfile and a clean diff.


Cleanup review by Oppenheimer | Re-run with /cleanup

@thurgood-bot

thurgood-bot Bot commented Jun 16, 2026

Copy link
Copy Markdown

Judicial Review

Dependency bump for Next.js and eslint-config-next to remediate reported vulnerabilities; no code-path or configuration changes observed.

No significant issues found. This code is cleared to ship.


Legal-Grade review by Thurgood | Re-review with /review

Comment thread bun.lock

"ajv-formats/ajv": ["ajv@8.17.1", "", { "dependencies": { "fast-deep-equal": "^3.1.3", "fast-uri": "^3.0.1", "json-schema-traverse": "^1.0.0", "require-from-string": "^2.0.2" } }, "sha512-B/gBuNg5SiMTrPkC+A2+cW0RszwxYmn6VYxB/inlBStS5nx6xHIt/ehKRhIMhqusl7a8LjQoZnjCs5vhwxOQ1g=="],

"browserslist/baseline-browser-mapping": ["baseline-browser-mapping@2.9.11", "", { "bin": { "baseline-browser-mapping": "dist/cli.js" } }, "sha512-Sg0xJUNDU1sJNGdfGWhVHX0kkZ+HWcvmVymJbj6NSgZZmW/8S9Y2HQ5euytnIgakgxN6papOAWiwDo1ctFDcoQ=="],

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WARNING: The lockfile now includes an extra alias entry browserslist/baseline-browser-mapping alongside the top-level baseline-browser-mapping. Repo-wide search shows only these lockfile entries; no source references. This may be a Bun lockfile artifact, but removing it could change dependency resolution or integrity.

Suggestion: Keep as-is unless you can regenerate bun.lock with the same Bun version and confirm the alias entry is unnecessary via bun install --frozen-lockfile and a clean diff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant