Merged
Conversation
* test: add comprehensive dashboard UI testing infrastructure Add Vitest component tests (110 tests across 13 files) covering utilities, Pinia stores, Vue components, and router configuration. Add Playwright E2E test scaffolding and integrate dashboard tests into CI pipeline. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: add empty changeset for testing infrastructure Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * ci: add Playwright E2E tests to GitHub Actions Add a separate CI job that builds the dashboard, installs Chromium, and runs Playwright E2E tests against a local wrangler dev server. Includes a dedicated E2E wrangler config without auth for CI compatibility. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: make E2E tests work by using worker dev directory for deps Move E2E worker entry and wrangler config to packages/worker/dev/ so worker dependencies (hono, chanfana, zod) resolve correctly. Fix app-loads test to use getByRole for strict element matching. Remove login E2E tests since auth is not configured in the E2E environment. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * test: add comprehensive E2E tests covering all dashboard features Add 35 Playwright E2E tests across 10 spec files covering: - File operations: create folder/file, delete, rename - File preview: text, JSON, markdown with filename header - File browsing: breadcrumbs, table columns, sorting - Navigation: folder drill-down, breadcrumb back, sidebar - Context menu: file vs folder menu items, open actions - Search: prefix search and clear - Metadata: open dialog, add custom metadata - Share links: create dialog, generate link, manage shares - Email: list with sender/subject, detail view, navigation Also adds API helpers (helpers.ts) for seeding test data via the worker upload/folder/delete endpoints. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * docs: add testing requirements to CLAUDE.md Require E2E test coverage for any UI changes and document test commands for component and E2E tests. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * test: expand E2E coverage with upload, download, preview, email, and share link tests Add 11 new E2E tests covering file upload/download, CSV/HTML preview, file edit/save/cancel, email body content, mark read/unread, share link options (expiration/password/max downloads), manage dialog, and revocation. Fix strict mode violations in app-loads and share-links tests. Set workers to 1 to prevent parallel race conditions on shared R2 bucket. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
* chore: include CHANGELOG.md and markdown files in docs package Add "files" field to docs package.json to ensure CHANGELOG.md and all .md files are included in the published package. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: include CHANGELOG.md in worker package, revert docs change Add CHANGELOG.md to the worker package "files" field so it's included in the published npm package. Revert the mistaken docs package change. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: copy CHANGELOG.md and docs markdown files into worker package Update the worker build script to copy CHANGELOG.md from root and all docs markdown files into a docs/ directory, so they are bundled in the published npm package. Add the generated paths to .gitignore. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
The build script was copying CHANGELOG.md from the repo root, but changesets generates it directly in packages/worker/. Removed the redundant copy that referenced a non-existent root file. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Mark docs-new and r2-explorer-dev-worker as private to prevent changesets from attempting to publish them to npm. Also add r2-explorer-dev-worker to the changeset ignore list. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )