Skip to content

docs: rule out IM-08, IM-11 and ST-15 (D52); 472 rows in scope - #21

Merged
siyabendoezdemir merged 1 commit into
mainfrom
lead/scope-d52
Oct 10, 2026
Merged

siyabendoezdemir merged 1 commit into
mainfrom
lead/scope-d52

Conversation

@siyabendoezdemir

@siyabendoezdemir siyabendoezdemir commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

D52 rules out three rows, because Orbit has no password manager or autofill store to import into. This PR marks them out of scope in the feature checklist and regenerates FEATURES.md. Docs only.

What changes

  • docs/feature-status.json: "scope": "out", with a note, for three rows:
    • IM-08 (password import): the spec has no row for a password manager (OT-11), and Electron has none. So IM-03 offers no Passwords type, and IM-12 and IM-13 take no passwords CSV.
    • IM-11 (address and card import): there's no autofill store (OT-12) to import into.
    • ST-15 (passwords and autofill settings): the settings of those two features.
  • FEATURES.md, regenerated with node scripts/features.mjs:
    • 472 rows in scope (was 475), and 13 out of scope (was 10);
    • P1 219 (was 221), P2 79 (was 80);
    • nothing built or tested changes.
  • The three rows come back if a password-manager spec is added (D52).

Merge gate (D42, D50)

Run locally on the Linux VPS on this PR's head, abe493c, which is main (e672076) plus this commit:

npm run build; npm run typecheck; npm run lint; ./node_modules/.bin/vitest run
  • Build, typecheck and lint: exit 0. Unit: 435 of 435 in 49 files.
  • E2E: carried over (D50).
  • macOS: unverified. Docs only.
  • Trial merges (git merge-tree) with lane/chrome and lane/downloads conflict only in FEATURES.md, which each lane regenerates after merging main.

Review

Independent review: APPROVE, no must-fix.

  • The three notes match spec/import.md, spec/settings.md and D52.
  • FEATURES.md is byte for byte what node scripts/features.mjs generates at abe493c.
  • Totals: 485 rows, 472 in scope and 13 out; P0 174, P1 219, P2 79.
  • Nothing else changed. Nothing in src, tests or the build reads either file, so D50's carry-over of the e2e result holds.

One follow-up, handled in the lead's docs outside this PR: ST-07's Passwords and Payment methods entries, and ST-20's mention of passwords and autofill, go with these three rows (D52, extended). Both rows stay in scope for their other bullets.

🤖 Generated with Claude Code


Note

Low Risk
Documentation and feature-tracking JSON only; no runtime, build, or test behavior changes.

Overview
Per D52, this PR records that Orbit has no password manager or autofill store (and Electron offers none), so three spec rows are out of scope instead of open backlog.

docs/feature-status.json adds "scope": "out" with rationale for IM-08 (password import), IM-11 (addresses/cards import), and ST-15 (passwords/autofill settings). FEATURES.md is regenerated via scripts/features.mjs: in-scope totals drop from 475 → 472 spec’d rows (13 out of scope, up from 10); P1/P2 in-scope counts adjust slightly. Built and tested totals are unchanged—documentation only, no app code.

Reviewed by Cursor Bugbot for commit abe493c. Bugbot is set up for automated code reviews on this repo. Configure here.

Orbit has no password manager or autofill store to import into: the spec has
no rows for them (OT-11, OT-12), and Electron has neither. ST-15 is their
settings. FEATURES.md regenerated: 472 spec'd, 13 out of scope.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@cursor

cursor Bot commented Oct 10, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: 1a2f289d-8a70-46fc-9766-48cb247db509)

@siyabendoezdemir
siyabendoezdemir merged commit 5e79fc5 into main Oct 10, 2026
1 check passed
siyabendoezdemir added a commit that referenced this pull request Oct 10, 2026
FEATURES.md regenerated from the merged feature status.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
siyabendoezdemir added a commit that referenced this pull request Oct 10, 2026
docs/feature-status.json merged key by key (IM-08, IM-11 and ST-15 from main); FEATURES.md regenerated.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant