Skip to content

chore: remove unused env var flags (PLAYWRIGHT_LEGACY_SCREENSHOT, PW_CODEGEN_NO_INSPECTOR, PW_DETECT_NESTED_PROGRESS)#41500

Open
faisalaffan wants to merge 1 commit into
microsoft:mainfrom
faisalaffan:chore-dead-env-vars
Open

chore: remove unused env var flags (PLAYWRIGHT_LEGACY_SCREENSHOT, PW_CODEGEN_NO_INSPECTOR, PW_DETECT_NESTED_PROGRESS)#41500
faisalaffan wants to merge 1 commit into
microsoft:mainfrom
faisalaffan:chore-dead-env-vars

Conversation

@faisalaffan

Copy link
Copy Markdown

Remove 3 env var flags that are never set anywhere in the repo, docs, or configs.

  • PLAYWRIGHT_LEGACY_SCREENSHOT — ternary defaulting to CDP flag; '' branch never executes
  • PW_CODEGEN_NO_INSPECTOR — two early-return guards in recorder; dead since Jul 2025 refactor
  • PW_DETECT_NESTED_PROGRESS — debug-only nested race detection block + cascading dead vars (outerProgress, allowConcurrent, setAllowConcurrentOrNestedRaces)

@faisalaffan

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

@faisalaffan faisalaffan changed the title chore: remove unused env var flags (PLAYWRIGHT_LEGACY_SCREENSHOT, PW_CODEGEN_NO_INSPECTOR, PW_DETECT_NESTED_PROGRESS) chore: remove unused env var flags (PLAYWRIGHT_LEGACY_SCREENSHOT, PW_CODEGEN_NO_INSPECTOR, PW_DETECT_NESTED_PROGRESS) Jun 27, 2026
…CODEGEN_NO_INSPECTOR, PW_DETECT_NESTED_PROGRESS)
@faisalaffan faisalaffan force-pushed the chore-dead-env-vars branch from bac49c5 to 21f3e7a Compare June 27, 2026 07:00
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