docs(status): correct HT-71 state, add marketplace pivot and inbox basics - #91
Merged
Conversation
…sics PR #83 landed already-stale: main moved by nine PRs while it was in review, and it shipped one claim that contradicts the charter. - HT-71 operator guide was listed as an open PR under Next. #81 merged at 17:48, ~2h before #83 merged at 19:39, and docs/modules/ has been on main since. Moved to Done with its precision follow-up (#84). - Marketplace was listed under 'Not yet / deferred'. CHARTER §3/§4/§5 were amended the same day (HT-79, #86) to make it a launch-day component of Phase 3 — built now, proven as the dogfood install path. Removed from deferred; marketplace v1 spec (#87, draft) now leads Next. - Added inbox basics (HT-76/77/78, #90, migrations 24-25): saved replies & macros, snooze, send & close — shipped engine features with no STATUS line. - Added catalog reclassification (HT-75, #82): KB and end-user portal are paid, 71-module gap audit closed, open-core line restated. Passkeys stay core, reconciled in #85. - Added passkey login spec (HT-75, specs/auth/passkeys.md, draft.3) to Next. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthrough
ChangesStatus roadmap refresh
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #83, which landed already-stale.
mainmoved by nine PRs while it was in review, and it shipped one claim that contradicts the charter.The two real defects in #83
1. HT-71 was listed as an open PR. #81 merged at
17:48Z— roughly two hours before #83 merged at19:39Z— anddocs/modules/has been onmainsince. STATUS was telling readers the operator guide didn't exist yet. Moved to Done, together with its precision follow-up #84 (event-transaction scope, payload sensitivity).2. The marketplace was listed under "Not yet / deferred." CHARTER §3/§4/§5 were amended the same day (HT-79, #86) to move the marketplace from "a later phase, once demand justifies it" to a launch-day component of Phase 3 — built now, proven first as Resonant IQ's own dogfood install path. STATUS was contradicting the constitution on a strategic question. Removed from deferred;
specs/modules/marketplace-v1.md(#87, draft for the maintainer review) now leads Next, with child tickets HT-81 (in-product directory) and HT-82 (dogfood-through-marketplace).Shipped work that had no STATUS line
saved_replies), snooze (conversation_snooze), and send & close.specs/auth/passkeys.md, draft.3, spec-only) — added to Next, not Done: no migrations, no implementation.Verification
Every claim checked against
mainat76a56ee— git log, PR merge timestamps,src/db/migrate.ts(migrations 24/25 confirmed by name), and the spec files' own status headers.mainwas re-confirmed unmoved immediately before push, which is the step that was missing last time.Unrelated observation, not fixed here
specs/modules/marketplace-v1.mdcarries a sequencing note saying the HT-79 charter amendment is "pushed but not yet merged tomainas of this draft" — it has since merged asb528971. That note is now stale and its citations are worth a re-check against the merged text, but it's outside this PR's scope.🤖 Generated with Claude Code
Summary by CodeRabbit