Skip to content

chore: Dependabot 보안 경고를 해소합니다 - #1063

Merged
jk-kim0 merged 1 commit into
mainfrom
jk/chore-dependabot-security-20260826
Aug 26, 2026
Merged

chore: Dependabot 보안 경고를 해소합니다#1063
jk-kim0 merged 1 commit into
mainfrom
jk/chore-dependabot-security-20260826

Conversation

@jk-kim0

@jk-kim0 jk-kim0 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Summary

기본 브랜치에서 열린 Dependabot 경고 15건(High 7, Moderate 7, Low 1)을 해소합니다.

문제 현상

  • GitHub Dependabot이 15개 advisory 경고를 보고합니다.
  • npm audit는 동일 advisory를 루트 6개 취약 패키지와 scripts/website-analysis 6개 취약 패키지로 묶어 표시합니다.
  • 루트에는 brace-expansion, dompurify, fast-uri, js-yaml, mermaid, nanoid 경고가 있습니다.
  • website analyzer에는 Puppeteer 24의 브라우저 다운로드 경로를 통한 extract-zip 등의 경고가 있습니다.

구현 내용

  • 루트 의존성 override를 각 advisory의 안전 버전 이상으로 갱신합니다.
  • js-yaml 직접 의존성을 4.3.1 이상으로 갱신합니다.
  • Puppeteer를 25.9.0으로 갱신해 취약한 @puppeteer/browsers 2.x 및 extract-zip 경로를 제거합니다.
  • Puppeteer 25의 공식 요구사항에 맞춰 website analyzer의 Node.js 최소 버전을 22.12로 조정합니다.
  • 두 npm lockfile을 재생성합니다.

Test plan

  • 루트 npm ci
  • scripts/website-analysis에서 PUPPETEER_SKIP_DOWNLOAD=true npm ci
  • 루트 및 website analyzer npm audit --package-lock-only — 각각 취약점 0건
  • npm run lint
  • npm run build
  • Puppeteer CommonJS import 및 headless Chrome 실행 smoke test
  • npm run test:run — 119건 중 117건 통과, origin/main에서도 동일하게 실패하는 internal spotlight 테스트 2건이 남아 있습니다.

Related tickets & links

🤖 Generated with Codex

## Summary
취약 버전으로 고정된 루트 override와 웹사이트 분석 도구의 Puppeteer 경로를 안전 버전으로 갱신합니다.

- js-yaml, Mermaid, DOMPurify 등 6개 루트 경고를 패치 버전으로 해소합니다.
- Puppeteer 25와 Node.js 22.12 이상을 사용해 취약한 브라우저 다운로드 경로를 제거합니다.
- 두 npm lockfile을 재생성해 audit 결과를 0건으로 맞춥니다.

Constraint: Puppeteer 25는 Node.js 22.12 이상과 ESM 기반 패키지 경로를 요구합니다.
Rejected: Puppeteer 24에 @puppeteer/browsers 3을 강제 override합니다. | 공식 의존성 조합이 아니므로 호환성을 보장할 수 없습니다.
Confidence: high
Scope-risk: moderate
Directive: website-analysis 실행 환경은 Node.js 22.12 이상을 유지합니다.
Tested: npm ci, npm audit, npm run lint, npm run build, Puppeteer import 및 headless Chrome smoke를 통과했습니다.
Not-tested: 전체 Vitest 중 origin/main에서도 실패하는 internal spotlight 테스트 2건은 통과하지 못했습니다.

🤖 Generated with Codex

Co-Authored-By: Atlas <atlas@jk.agent>
@vercel

vercel Bot commented Aug 26, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
querypie-docs Ready Ready Preview Aug 26, 2026 2:09pm

Request Review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3f758514b4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

},
"engines": {
"node": ">=18.0.0"
"node": ">=22.12.0"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Update the analyzer's documented Node requirement

When developers follow scripts/website-analysis/README.md, they are still told that Node.js 18+ is supported, but this change raises the analyzer and Puppeteer requirement to Node.js 22.12+. Node 18 users may therefore receive EBADENGINE, fail installation when engine-strict is enabled, or encounter unsupported runtime behavior; update the README requirement alongside this engine change.

AGENTS.md reference: AGENTS.md:L7-L9

Useful? React with 👍 / 👎.

@jk-kim0
jk-kim0 merged commit a1c10dd into main Aug 26, 2026
5 checks passed
@jk-kim0
jk-kim0 deleted the jk/chore-dependabot-security-20260826 branch August 26, 2026 14:19
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