Projects: workspace item-removal UX (projects-005)#35
Open
jaeyunha wants to merge 1 commit into
Open
Conversation
Make the browser smoke seed and open the dedicated Projects workspace fixture before exercising the side-panel lifecycle. Preserve route-visible archive/remove confirmations so the acceptance path is observable after navigation. Constraint: controller rejected the previous QA pass because the env-loaded system-Chrome side-panel gate timed out on the acceptance path. Rejected: relying on the organization projects list to discover a workspace | clean .env.test seeding can render Projects 0 unless the projects workspace fixture is explicitly enabled. Confidence: high Scope-risk: narrow Directive: keep projects item lifecycle E2E tied to PROJECTS_WORKSPACE_E2E and the returned projectsWorkspaceHref; do not replace it with a mocked or list-discovery pass. Tested: make doctor; ./hack/cargo_locked.sh test -p opengithub-api --test projects_workspace_contract -- --nocapture; make check; cd web && npx vitest run tests/project-workspace-page.test.tsx; cd web && set -a; . ../.env.test; set +a; PLAYWRIGHT_CHROMIUM_EXECUTABLE_PATH=/usr/bin/google-chrome CARGO_TARGET_DIR="/home/jaeyunha/dev/namuh/opengithub-visible/projects-005-visible-retry-20260512233455/../.scratch/cargo-target" npx playwright test tests/e2e/projects-items-side-panel.spec.ts --project=chromium Not-tested: full make test wrapper; deterministic required subcommands passed.
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Project workspace item-removal UX (notice-on-redirect) and scope-aware archived-items workspace href.
—
Preserved from QA-worktree cleanup on 2026-06-16. Web QA: biome lint clean; vitest green where unit tests exist. Rust build/test not run locally (would regenerate large target dirs). Opened against
stagingfor review.🤖 Generated with Claude Code