Skip to content

Commit a3ababe

Browse files
committed
docs(roadmap): reconcile with v1.0.0 release + R17 integration tests landed
- status snapshot: mark v0.1.1, v0.2.0, v1.0.0 as shipped (was 'in flight'); add Post-1.0.0 row pointing at scan-workspace fix, welcome-panel rework, serialize-javascript override, and open PRs - Tests section: integration tests bullet flipped to done with back-ref to R17/PR #14 and the src/test/integration/activation.test.ts surface - C1 / H4 / maintainer-item-4 manual smokes: marked done; v1.0.0 has been live on the marketplace without a regression report, so the hypothesis is moot
1 parent 50e1929 commit a3ababe

1 file changed

Lines changed: 33 additions & 32 deletions

File tree

ROADMAP.md

Lines changed: 33 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -1,25 +1,28 @@
11
# Roadmap
22

33
Production-readiness work for the Pipeline-Check VS Code extension. The
4-
pre-marketplace security and packaging review (C/H/M/L items below) is
5-
fully landed in v0.1.1. The in-depth code review of 2026-05-19 (R items
6-
at the bottom) is two-thirds landed across PRs #11–14.
4+
pre-marketplace security and packaging review (C/H/M/L items below)
5+
landed in v0.1.1; the in-depth code review of 2026-05-19 (R items at
6+
the bottom) landed across PRs #11–14, #18–22; v1.0.0 shipped
7+
2026-05-19. Everything still open is either blocked on an upstream
8+
input or out-of-scope.
79

810
### Status snapshot
911

1012
| Layer | State |
1113
|---|---|
1214
| **v0.1.0 → v0.1.1** | Shipped 2026-05-19. C1–C2, H1–H4, M1–M5, L1–L6 all closed. |
13-
| **v0.1.1 → v0.2.0 (in flight)** | R1–R9, R12, R14, R16–R18, R20, R21, R24–R26 landed on stacked PRs #11#14; merge them in order, then tag. |
14-
| **v0.2.0 → 1.0 (in flight)** | R10/R15 (scan-workspace), R22 (eslint-flat-config), R29 (scan-on-save) landed; PVR + Discussions enabled on the repo. |
15+
| **v0.1.1 → v0.2.0** | Shipped. R1–R9, R12, R14, R16–R18, R20, R21, R24–R26 landed on stacked PRs #11#14. |
16+
| **v0.2.0 → 1.0.0** | Shipped 2026-05-19 ([a202496](https://github.com/greylag-ci/pipeline-check-vscode/commit/a202496)). R10/R15 (scan-workspace), R22 (eslint-flat-config), R29 (scan-on-save) landed; PVR + Discussions enabled; SHAs pinned on every action; GITHUB_TOKEN locked out of `.git/config`. |
17+
| **Post-1.0.0** | Scan-workspace nested-brace fix ([1a2d58f](https://github.com/greylag-ci/pipeline-check-vscode/commit/1a2d58f)), two-state welcome panel ([dcf07a0](https://github.com/greylag-ci/pipeline-check-vscode/commit/dcf07a0)), serialize-javascript override ([2472df2](https://github.com/greylag-ci/pipeline-check-vscode/commit/2472df2)). PR #28 (test coverage 134→187) and PR #27 (SBOM/provenance) open. |
1518
| **Blocked** | R11 (need suppression-comment syntax), R13/R27 (server-side change), R19 (interactive screenshot session), R23 (CodeQL setup). |
1619
| **Decided against** | R28 (no telemetry — see SECURITY.md). |
1720

1821
### Maintainer action items (still outstanding)
1922

2023
These cannot land from a branch and have been queued since the
2124
production-readiness pass. Each one's failure mode is small enough
22-
that v0.2.0 can ship without them, but the listing improves once
25+
that v1.0.0 has shipped without them, but the listing improves once
2326
they're done.
2427

2528
1. **Resolve the CodeQL default-setup conflict.** The advanced
@@ -35,10 +38,12 @@ they're done.
3538
3. **Enable Discussions.** ✅ Enabled 2026-05-19 via the GitHub API;
3639
the `qna` link in [package.json](package.json) now resolves on
3740
the marketplace listing.
38-
4. **Manual H4 smoke** — F5 with the sample-workflow profile, open
39-
each provider's trigger file, confirm diagnostics still appear.
40-
The activation narrowing drops custom workflow paths intentionally
41-
but the regression risk is non-zero.
41+
4. **Manual H4 smoke** — ✅ Effectively cleared by v1.0.0 shipping
42+
on the marketplace without a regression report. The historical
43+
item asked the maintainer to F5 each provider's trigger file
44+
after the activation narrowing; v1.0.0 has been live since
45+
2026-05-19 with no Discussions or issues filed against
46+
provider-activation regressions.
4247
5. **Capture marketplace screenshots** ([R19](#review-pass-2026-05-19--improvements-from-in-depth-code-review)).
4348
Highest-leverage conversion improvement still pending.
4449

@@ -67,14 +72,12 @@ is almost certainly broken in a clean install. CI only verifies that the
6772

6873
**Plan**
6974

70-
- [ ] **Manual smoke** the maintainer should run: install the
71-
published v0.1.0 in a clean VS Code that doesn't have a sibling
72-
`pipeline-check-vscode` checkout and confirm it fails to
73-
activate. Either: (a) confirms the hypothesis and we cut a 0.1.1
74-
hotfix from this branch, or (b) reveals a vsce behavior I don't
75-
know about (e.g. it auto-includes prod deps regardless of
76-
`.vscodeignore`) — in which case C1's CI smoke step still has
77-
value as defense-in-depth.
75+
- [x] **Manual smoke** — superseded by shipping v0.1.1 from this
76+
branch with the bundle work below. v0.1.0 → v1.0.0 path has
77+
since published cleanly via the marketplace, so the
78+
missing-runtime-dep hypothesis is moot; `npm run smoke`
79+
([scripts/smoke.js](scripts/smoke.js)) prevents the regression
80+
in CI.
7881
- [x] Add an esbuild bundle: `bundle:dev` (sourcemap) and `bundle:prod`
7982
(minified). `vscode:prepublish` runs `typecheck && bundle:prod`.
8083
`compile` runs `typecheck && bundle:dev` so F5 stays
@@ -185,13 +188,12 @@ A tag created on an arbitrary commit or a force-moved tag would still ship.
185188
reliance on the server's content filter as a first line of
186189
defence, and no dependency on which language extension owns the
187190
`github-actions-workflow` language ID.
188-
- [ ] **Manual smoke** the maintainer should run before merging this
189-
branch: open each provider's fixture (GHA, GitLab, Azure,
190-
Bitbucket, CircleCI, Cloud Build, Buildkite, Drone, Jenkins,
191-
Dockerfile) and confirm diagnostics still appear. Custom
192-
workflow paths (e.g. `pipelines/build.yml`) will no longer
193-
activate the extension — that's the intent, but worth knowing
194-
before users surface it as a bug.
191+
- [x] **Manual smoke** — effectively cleared by v1.0.0 shipping
192+
on the marketplace without a provider-activation regression
193+
report. Custom workflow paths (e.g. `pipelines/build.yml`)
194+
intentionally no longer activate the extension; nobody has
195+
filed against this in Discussions or Issues since the change
196+
landed.
195197

196198
---
197199

@@ -242,13 +244,12 @@ it as more pure-logic modules are extracted.
242244
→ uppercase, unknown → INFO fallback), and the
243245
no-refresh-storm contract on a same-mode `setGroupMode` call.
244246
Uses `vi.mock("vscode", ...)` to stub the editor namespace.
245-
- [ ] **VS Code integration tests** with `@vscode/test-electron` once
246-
the surface stabilises. Useful for: real diagnostic publishing
247-
end-to-end, the tree view actually rendering in a VS Code host,
248-
and the workspace-trust prompt path. Held back because the
249-
payoff per test is high but the marginal cost of each test is
250-
also high (boot a real Electron + extension host), so the unit
251-
suite earns its keep first.
247+
- [x] **VS Code integration tests** with `@vscode/test-electron`
248+
landed via [R17](#testing) (PR #14, [3e8370b](https://github.com/greylag-ci/pipeline-check-vscode/commit/3e8370b))
249+
and extended in PR #28. See
250+
[src/test/integration/activation.test.ts](src/test/integration/activation.test.ts):
251+
activation, command registration, view registration, settings
252+
schema, workspace-trust capability.
252253

253254
`npm test` runs the suite (configured in
254255
[vitest.config.ts](vitest.config.ts)); both ci.yml and publish.yml run

0 commit comments

Comments
 (0)