From aa5107146337c0bce69905b5d881dcba4cfd7239 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=98=D0=BB=D0=B8=D1=8F?= Date: Fri, 23 Oct 2026 22:33:23 +0300 Subject: [PATCH] test(cursor): include composed project lifecycle in all platform gates --- .../cursor_project_lifecycle_test.go | 2 +- .../cursor-code-review-r1.md | 18 ++++++++++++++++++ .../cursor-code-review-r2.md | 4 ++++ .../cursor-implementation-evidence.md | 10 +++++++++- 4 files changed, 32 insertions(+), 2 deletions(-) diff --git a/cli/cmd/agentplugins/cursor_project_lifecycle_test.go b/cli/cmd/agentplugins/cursor_project_lifecycle_test.go index dd58daae2..71b2f2080 100644 --- a/cli/cmd/agentplugins/cursor_project_lifecycle_test.go +++ b/cli/cmd/agentplugins/cursor_project_lifecycle_test.go @@ -11,7 +11,7 @@ import ( // This is the production CLI/dependency/installer/native-filesystem composition // with real loopback TLS Git and ZIP acquisition. Native Cursor discovery, // enablement, MCP initialization, model use and runtime are not certified. -func TestCursorProjectComposedSourceLifecycle(t *testing.T) { +func TestProjectCursorComposedSourceLifecycle(t *testing.T) { for _, kind := range []string{"git", "zip"} { t.Run(kind, func(t *testing.T) { transport := newProjectJourneyTransport(t, kind) diff --git a/docs/plans/project-install-e2e/cursor-code-review-r1.md b/docs/plans/project-install-e2e/cursor-code-review-r1.md index 7ce369d80..bfd68b564 100644 --- a/docs/plans/project-install-e2e/cursor-code-review-r1.md +++ b/docs/plans/project-install-e2e/cursor-code-review-r1.md @@ -94,3 +94,21 @@ client run remains recorded above. No blocker remains in this correction. Neither review execution launched Cursor, its launcher or an MCP process, and the genuine hosted native boundary is unchanged. The independent R2 author has separately recorded GO for the same corrective checkpoint. + +## Scoped hosted test-selection closure: GO at 32c33c6f + +Independently reviewed `32c33c6fa96d790396a6fc1be3f1a36537ef4c9c` +after the reviewed project-stack CI baseline merge `935e3e08`. The complete +source file is byte-identical after replacing only the function symbol +TestCursorProjectComposedSourceLifecycle with +TestProjectCursorComposedSourceLifecycle; no fixture or assertion changed. +The adjacent evidence accurately updates its test name and records pending +hosted/native results. + +Extracted the actual project-contracts workflow regex +`Test(TargetContext|Project|PrepareRejectsCrossEnvironmentContext)` and checked +that it selects the corrected name while excluding the previous spelling. +That job includes the public CLI package on Linux/macOS/Windows. No broad +compile or rerun is needed for this literal symbol-only correction, and none +was represented as fresh execution. The bounded selection correction is GO; +real cross-platform/native outcomes remain their separate hosted gates. diff --git a/docs/plans/project-install-e2e/cursor-code-review-r2.md b/docs/plans/project-install-e2e/cursor-code-review-r2.md index fb319eb1c..a9206941e 100644 --- a/docs/plans/project-install-e2e/cursor-code-review-r2.md +++ b/docs/plans/project-install-e2e/cursor-code-review-r2.md @@ -79,3 +79,7 @@ trust/approval/login change or hosted native proof was executed. Author's corrective lint gates and the other independent review remain applicable to the final checkpoint. Actual native discovery, fresh native restarts, editor behavior, MCP handshake/tool invocation and release qualification remain pending. +# Hosted selector closure — 2026-10-23 + +Independent R2 rechecked isolated `32c33c6f` against its parent: the only test-code change renames `TestCursorProjectComposedSourceLifecycle` to `TestProjectCursorComposedSourceLifecycle`. The entire Git/ZIP lifecycle body and assertions are unchanged. The existing cross-platform project-contract selector `Test(TargetContext|Project|PrepareRejectsCrossEnvironmentContext)` now includes the composed test through its `TestProject` prefix. The accompanying evidence note correctly leaves pending hosted/native runs pending. **GO for this symbol-only selection correction.** No additional compilation was needed to establish this exact rename preservation; previously reviewed composed behavior retains its prior evidence. + diff --git a/docs/plans/project-install-e2e/cursor-implementation-evidence.md b/docs/plans/project-install-e2e/cursor-implementation-evidence.md index 765b8bb12..34b2c4d1d 100644 --- a/docs/plans/project-install-e2e/cursor-implementation-evidence.md +++ b/docs/plans/project-install-e2e/cursor-implementation-evidence.md @@ -120,7 +120,7 @@ implementations. Only host detection and package security assessment are inert fixture boundaries. Remote MCP entries avoid executing configured commands or copying the test executable as a managed launcher. -`TestCursorProjectComposedSourceLifecycle` covers both Git and ZIP sources: +`TestProjectCursorComposedSourceLifecycle` covers both Git and ZIP sources: saved project add in A and B, exact native skill/MCP paths, locked no-op sync, list/info/doctor/outdated/dependency status/tree reads without native/state changes, missing native skill repair, pin-advancing update that retains binding @@ -164,3 +164,11 @@ the positive read-only case, then requires wrong/empty data locators to fail with unchanged immutable package bytes. Full project Cursor provider selection passes at this correction: **PASS 0.180s**. The independent R2 reviewer must recheck this correction before closing the adapter review. + +## Cross-platform source journey selection + +The composed Git/ZIP source lifecycle test now starts with `TestProjectCursor` +so the existing project-contracts selector includes it on Linux, macOS and +Windows. Only the test name changes; its assertions and inert source/runtime +boundaries remain identical. This corrects hosted test selection and does not +claim that any pending native or cross-platform run has passed.