Skip to content

revert(read)!: unwind read-shape doctrine PRs 1–5 (#375–#379) pending re-audit - #398

Merged
mikegreiling merged 6 commits into
mainfrom
mg/revert-read-shape-doctrine
Aug 5, 2026
Merged

revert(read)!: unwind read-shape doctrine PRs 1–5 (#375–#379) pending re-audit#398
mikegreiling merged 6 commits into
mainfrom
mg/revert-read-shape-doctrine

Conversation

@mikegreiling

Copy link
Copy Markdown
Owner

Reverts the five read-shape doctrine PRs — #375, #376, #377, #378, #379 — as five clean revert commits applied in reverse order (PR 5 → PR 1), plus a follow-up bookkeeping commit.

What this does

  • Restores the pre-0.13.0 bucketed JSON read shapes:
    • todaydata.sections[] (Today / This-Evening) + data.badge {dueOrOverdue, other} (the "badge" vocabulary is un-purged; the flat items[] is gone).
    • project show / get_project → per-stage child buckets (anytime / upcoming / someday), {heading: node} groups, and a separate logbookHeadings alongside the flat logbook.
    • area show / get_area → direct-to-do stage buckets (projects[] unchanged).
    • meta.counts is removed.
  • Breaking (ALPHA-CONTRACT): a clean revert — no compatibility shims, no dual shapes. The next release must be 0.14.0.
  • The unrelated 0.13.0 changes (flat container refs, projectIsTemplate, heading-archived node) are not affected.

Re-audit input retained

docs/design/read-shape-doctrine.md is retained (the PR-1 revert deleted it; it was restored) with its status set to REVERTED (dated) — it is the input to a design re-audit of the read-shape approach, not deleted.

Conflict handling

Every revert conflicted only in CHANGELOG.md at the ## Unreleased / ## 0.13.0 boundary (the doctrine entries now live in the released 0.13.0 history section). Each was resolved by keeping the current structure — the released 0.13.0 (and earlier) sections are immutable history and were left untouched; a single breaking revert entry was added under ## Unreleased.

Follow-ups

  • skills/things-cli/SKILL.md — the npx floor sentence was reconciled: the skill again teaches the bucketed read shapes (which 0.13.0 does not have; they return in the next release), so the floor is anchored to the skill's publish-stamped version rather than the now-false "below 0.13.0" literal.
  • PR feat(cli): de-gutter homogeneous container listings (TTY only) #368 (the open de-gutter PR) will need another rebase after this lands.

🤖 Generated with Claude Code

https://claude.ai/code/session_01C1g2PkB43exMAfFdg6QnMT

mikegreiling and others added 6 commits August 4, 2026 20:57
…(read-shape doctrine PR 4) (#378)"

This reverts commit 044ffd0.
…read-shape doctrine PR 3) (#377)"

This reverts commit 94f1dda.
…e purge (read-shape doctrine PR 1) (#375)"

This reverts commit ce4fa4d.
…ill floor

Follow-up to the five read-shape doctrine reverts (#375-#379):

- CHANGELOG: add a breaking Unreleased entry documenting the wholesale
  revert and the return to pre-0.13.0 bucketed today/project/area shapes;
  meta.counts removed. Clean revert, no compat shims (ALPHA-CONTRACT).
- docs/design/read-shape-doctrine.md: restored (the PR-1 revert deleted it)
  with its status changed to REVERTED (dated) — retained as the input to a
  design re-audit of the read-shape approach.
- skills/things-cli/SKILL.md: reconcile the npx floor sentence. The skill
  again teaches the bucketed read shapes, which 0.13.0 does NOT have; anchor
  the floor to the skill's publish-stamped version (the release that ships
  these shapes) instead of the now-false "below 0.13.0" literal.

Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01C1g2PkB43exMAfFdg6QnMT
@mikegreiling
mikegreiling merged commit 6329f98 into main Aug 5, 2026
1 check passed
@mikegreiling
mikegreiling deleted the mg/revert-read-shape-doctrine branch August 5, 2026 02:05
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