Branch: feat/migrate-to-playwright-core-packages
Priority: High
Estimate: 2-3 hours
Parent Epic: #1
Dependencies: #2, #3, #4
Description
Update lytics-playwright repository to consume packages from the @lytics/playwright-core monorepo instead of using internal reporter implementation. This completes the extraction and demonstrates that the new packages work as drop-in replacements with no loss of functionality.
Acceptance Criteria
Technical Requirements
- Install
@lytics/playwright-annotations, @lytics/playwright-reporter, @lytics/playwright-adapters-internal
- Update
reporter.ts to import and use CoreReporter from package
- Update
slack-reporter.ts to use SlackAdapter (if migrated)
- Remove
src/reporter/ directory entirely
- Remove
src/types/core.ts file
- Update all test imports (if any reference reporter internals)
- Run full regression suite to validate
- Update CI/CD to authenticate with GitHub Packages (for internal packages)
- Update
README.md and internal docs
Notes
- This is a breaking change for
lytics-playwright but isolated to that repo
- Should be done after all core packages are published and stable
- Consider doing thorough QA in staging environment before production
Branch:
feat/migrate-to-playwright-core-packagesPriority: High
Estimate: 2-3 hours
Parent Epic: #1
Dependencies: #2, #3, #4
Description
Update
lytics-playwrightrepository to consume packages from the@lytics/playwright-coremonorepo instead of using internal reporter implementation. This completes the extraction and demonstrates that the new packages work as drop-in replacements with no loss of functionality.Acceptance Criteria
lytics-playwrightrepositoryTechnical Requirements
@lytics/playwright-annotations,@lytics/playwright-reporter,@lytics/playwright-adapters-internalreporter.tsto import and useCoreReporterfrom packageslack-reporter.tsto useSlackAdapter(if migrated)src/reporter/directory entirelysrc/types/core.tsfileREADME.mdand internal docsNotes
lytics-playwrightbut isolated to that repo