Skip to content

MP-122: Статус merged в свежей базе mypi - #89

Merged
prineycom merged 4 commits into
mainfrom
task/MP-122-merged-status-fresh-db
Nov 11, 2026
Merged

prineycom merged 4 commits into
mainfrom
task/MP-122-merged-status-fresh-db

Conversation

@prineycom

Copy link
Copy Markdown
Collaborator

Goal

A database created by pnpm bootstrap had no merged status, so mp-task status <KEY> merged failed and /ship left tasks in review. The fresh seed now includes merged (non-terminal); existing databases are not changed (schema stays 15, no db init).

Plan: https://draft.yokeloop.com/artifacts/mp-122-merged-mypi-r8241gm5

Steps

  1. Seed ('merged', 0) last in 001-initial.sql (id 11; ids 1–10 unchanged).
  2. test/fast/database.test.ts: fresh status list expects merged; the no-op-on-current-database part now holds merged with another id.
  3. New test: a fresh database lets a task go review → merged → done.
  4. CHANGELOG: ### Fixed entry; older databases add the status once with mypi status add merged.

Checks

  • mise exec -- pnpm build — exit 0
  • mise exec -- pnpm verify — exit 0 (fast 237/237, boundary 67/67)

Stand acceptance (disposable, XDG_STATE_HOME under a scratch dir)

  1. Bootstrap (pseudo-TTY, answer y) → status list ends with {"id":11,"code":"merged","isTerminal":false}.
  2. Fixture task FX-1: review → merged (exit 0, task list shows merged) → done (exit 0).
  3. Copy of the stand DB (schema 15, with custom added and paused renamed hold): status list and task list before and after db init — diff empty; status list still opens it.

Review

task-reviewer: approved, no findings.

Note: commit 027515f alone leaves the fast suite red until 6642bd0 updates the expectation (separate plan steps).

🤖 Generated with Claude Code

@prineycom
prineycom merged commit 2e060a8 into main Nov 11, 2026
1 check passed
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