Revert PER-9560 CI changes — restore original test/windows workflows

[AkashBrowserStack]

Summary

Reverts the PER-9560 CI changes made this session — restores .github/workflows/test.yml and windows.yml to their pre-PER-9560 state.

Net diff is just removing the paths-ignore block from both workflows (added in #2293):

   pull_request:
-    paths-ignore:
-      - 'lerna.json'
-      - 'packages/*/package.json'
   workflow_dispatch:

#2293 had already reverted #2284's machinery (the changes job, the version_only / github-actions[bot] gate, and the permissions block), so removing the path filter restores the original workflows exactly.

Effect: version-bump PRs will once again run the full Linux + Windows test suites — i.e., PER-9560 is fully backed out. No other behavior change.

Reverts

• ci: skip heavy test suites on bot version-bump PRs (PER-9560) #2284 — ci: skip heavy test suites on bot version-bump PRs
• ci: skip heavy suites on version-bump PRs via paths-ignore (PER-9560) #2293 — ci: skip heavy suites on version-bump PRs via paths-ignore

🤖 Generated with Claude Code

[AkashBrowserStack]

🤖 Claude Code Review — stack:pr-review

PR: #2298 • Head: 0a8fc03 • Reviewers: stack:percy-cli-code-reviewer (harness percy-cli stack) + inline 18-row checklist

Summary: Pure −6-line revert — removes the on.pull_request.paths-ignore block (lerna.json, packages/*/package.json) from test.yml and windows.yml, restoring them to their pre-PER-9560 state. Backs out the PER-9560 work (#2284 → reverted by #2293 → path filter removed here); version-bump PRs once again run the full Linux + Windows suites. Deletion-only, no additions.

Priority	Category	Check	Status	Notes
High	Security	No hardcoded secrets / credentials	✅ Pass	Deletions only; pre-existing secrets.PERCY_REGRESSION_TOKEN untouched
High	Security	Authn/authz checks present	N/A	Workflow config
High	Security	Input validation / sanitization	✅ Pass	Nothing added; no injection surface
High	Security	No IDOR	N/A	—
High	Security	No SQL injection	N/A	—
High	Correctness	Logic correct, edge cases handled	✅ Pass	Only the 3-line paths-ignore removed per file; build/test/regression jobs, matrix, and PER-9011 retry logic unchanged; effect is stricter CI (full suite runs again), not looser
High	Correctness	Explicit error handling	N/A	No runtime logic changed
High	Correctness	No race/concurrency issues	N/A	—
Medium	Testing	New code has tests	N/A	CI config — restores a previously-shipped state
Medium	Testing	Error/edge paths tested	N/A	—
Medium	Testing	Existing tests still pass	✅ Pass	No test logic changed
Medium	Performance	No N+1 / unbounded fetch	N/A	—
Medium	Performance	Long tasks backgrounded	N/A	—
Medium	Quality	Follows codebase patterns	✅ Pass	Restored on: block matches standard repo trigger style; valid YAML
Medium	Quality	Focused (single concern)	✅ Pass	Two workflow files, deletions only
Low	Quality	Meaningful names, no dead code	✅ Pass	Removes the path filter cleanly; no remnants
Low	Quality	Comments explain why	✅ Pass	N/A — no comments involved
Low	Quality	No unnecessary deps	N/A	—

Findings: No blocking findings. Scope verified — git diff --name-status shows only the two workflow files (6 deletions, 0 additions); no paths-ignore/version_only/changes:/permissions: remnants.

---

Verdict: PASS