Skip to content

Conversation

@romradchenko
Copy link
Owner

This PR finalizes the @playwright-kit/auth v0.2.0 release and tightens DX around typing and CI/release hygiene.

Breaking changes

Removed baseTest / baseExpect from authTest() and createAuthTest() to keep the wrapper strictly aligned with Playwright’s default test types.

Changes

DX/strictness: added type-level regression coverage to ensure Playwright option types remain intact with authTest() (authTest-use-options-type.test.ts).
CLI: removed --fail-fast (was only in help, not documented in README) and simplified ensure behavior to always process all selected profiles.
.env: restored dotenv-based loading for --dotenv / --dotenv-path and updated docs/help accordingly.
Repo hygiene: lefthook pre-commit runs lint + typecheck, pre-push runs full check.
Added a GitHub Action + release notes doc for publishing @playwright-kit/auth via tags (release-auth.yml, RELEASING.md).

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR finalizes the @playwright-kit/auth v0.2.0 release with breaking changes that simplify the API and improve type safety, while also adding essential release infrastructure.

  • Removed baseTest and baseExpect options to keep authTest() strictly aligned with Playwright's default types
  • Removed undocumented --fail-fast CLI flag to ensure deterministic ensure behavior
  • Added type-level regression tests to verify Playwright option types remain intact

Reviewed changes

Copilot reviewed 14 out of 15 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/auth/type-tests/authTest-use-options-type.test.ts New type test verifying Playwright options and fixtures are properly typed when using authTest()
packages/auth/src/fixtures/createAuthTest.ts Removed baseTest/baseExpect parameters from both authTest() and createAuthTest() functions, hardcoded to Playwright defaults
packages/auth/src/cli/main.ts Removed failFast parameter when calling authEnsure command
packages/auth/src/cli/help.ts Removed --fail-fast from usage documentation and updated dotenv notes
packages/auth/src/cli/commands/authEnsure.ts Removed failFast option and early-exit logic from ensure loop
packages/auth/src/cli/args.ts Removed failFast field from ParsedArgs type and parsing logic for --fail-fast flag
packages/auth/src/tests/argParsing.test.ts Updated test expectation to remove failFast field
packages/auth/package.json Version bumped from 0.1.2 to 0.2.0
package-lock.json Updated lockfile to reflect new package version
packages/auth/README.md Updated dotenv documentation to clarify it's provided via the dotenv package
packages/auth/CHANGELOG.md Added 0.2.0 release notes documenting breaking changes
lefthook.yml Configured pre-commit to run lint and typecheck in parallel, pre-push to run full check
RELEASING.md New documentation explaining the release process for @playwright-kit/auth
README.md Added reference to RELEASING.md and updated dotenv notes
.github/workflows/release-auth.yml New GitHub Actions workflow for publishing package on tag push

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@romradchenko romradchenko merged commit 45ddec9 into main Dec 30, 2025
9 checks passed
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