Skip to content

chore: remove stale DEPS.list entries and add validation tooling#40157

Merged
pavelfeldman merged 1 commit intomicrosoft:mainfrom
pavelfeldman:chore-cleanup-stale-deps
Apr 10, 2026
Merged

chore: remove stale DEPS.list entries and add validation tooling#40157
pavelfeldman merged 1 commit intomicrosoft:mainfrom
pavelfeldman:chore-cleanup-stale-deps

Conversation

@pavelfeldman
Copy link
Copy Markdown
Member

Summary

  • Remove ~110 stale entries across 38 DEPS.list files (broken @playwright/ scope refs, removed files/directories, unused node_modules/ allowlists, type-only imports that don't need DEPS entries)
  • Add utils/build_deps_true.js — generates DEPS.true ground-truth files from actual imports
  • Add utils/validate_deps.js — compares DEPS.list against DEPS.true and reports unused entries
  • Add DEPS.true to .gitignore

Add build_deps_true.js and validate_deps.js scripts to detect declared
dependencies in DEPS.list files that are not actually used by any source
file. Remove ~110 stale entries found across 38 DEPS.list files.
@github-actions
Copy link
Copy Markdown
Contributor

Test results for "tests 1"

6 flaky ⚠️ [chromium-library] › library/popup.spec.ts:261 › should not throw when click closes popup `@ubuntu-22.04-chromium-tip-of-tree`
⚠️ [chromium-library] › library/video.spec.ts:342 › screencast › should work for popups `@ubuntu-22.04-chromium-tip-of-tree`
⚠️ [chromium-library] › library/beforeunload.spec.ts:130 › should support dismissing the dialog multiple times `@chromium-ubuntu-22.04-arm-node20`
⚠️ [chromium-library] › library/video.spec.ts:342 › screencast › should work for popups `@chromium-ubuntu-22.04-node20`
⚠️ [firefox-page] › page/page-emulate-media.spec.ts:144 › should keep reduced motion and color emulation after reload `@firefox-ubuntu-22.04-node20`
⚠️ [webkit-library] › library/har.spec.ts:164 › should include form params `@webkit-ubuntu-22.04-node20`

39154 passed, 846 skipped


Merge workflow run.

@github-actions
Copy link
Copy Markdown
Contributor

Test results for "MCP"

1 failed
❌ [chrome] › mcp/cli-session.spec.ts:124 › delete-data non-existent session @mcp-macos-latest

6480 passed, 383 skipped


Merge workflow run.

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.

2 participants