Repository navigation
fix(relationship-tracker): 0.6.3 match the built-in Tracker Panel sections - #1248
SpicyMarinara merged 1 commit into
Conversation
…tions Marinara's August UI pass (139e481 here, Engine #5509) gave package tracker sections the same collapsible header as the built-in trackers. Relationship Tracker joined the catalog later, so it still drew its own card with a large title, which a user reported on Engine 2.5.0. - Tracker Panel: the section uses the built-in header (chevron, icon, uppercase title), drops its own card, remembers being collapsed, and shows a spinner while the tracker runs. Collapsing does not re-render, so unsaved editor text survives. - Desktop Trackers window: the host drawer already shows the title and arrow, so the package header is left out there. - The toolbar tile wears the host's toolbarButtonClass like Quartermaster, Beholder and Memory Nag; presses pass through to the drawer header. Runtime payloads come from the Relationship Tracker source tree, where 172 regression tests pass. No Marinara Engine changes; the Engine range and Capability API are unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
⛔ Files ignored due to path filters (2)
📒 Files selected for processing (9)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughRelationship Tracker is updated to version 0.6.3. Package and catalog metadata record the updated client and archive details. Release notes describe Tracker Panel collapse behavior and desktop Trackers window UI changes. ChangesRelationship Tracker release
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The 0.6.3 release records are consistent and no actionable merge risk remains. Proceed with normal staging checks. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Pull request
Important
Contributions target
staging. OnlySpicyMarinaramay promote this repository'sstagingbranch tomain.Outside and first-time contributors also require an approving review from
SpicyMarinara.Linked issue
Closes #1247
Why this change
What changed
relationship-tracker0.6.2 → 0.6.3 (patch; CHANGELOG entry without a highlight).SectionHeader(chevron, icon, uppercase title, with the same spacing, divider and constrained-width text scale) and no longer draws its own card; the host's section shell supplies the background and divider. It collapses like the built-in sections, remembers that across reloads (localStorage keymarinara.relationship-tracker.trackerPanelCollapsed), and toggles in place, so unsaved editor text survives. While the tracker runs, the header shows a small spinner where the built-in rows spin their re-run icon..mari-drawer__body, a documented theming hook.toolbarButtonClass, as Quartermaster, Beholder and Memory Nag do, instead of a self-styled↔. It is slotted into light DOM so the host classes apply, has no action of its own, and lets presses through, so in the Trackers window a press opens the drawer like the built-in previews. Engines that pass no class get the previous self-styled mark, now with the same icon.artifacts/relationship-tracker-0.6.3.zip(206,379 bytes, SHA-2565e3d6fd318bfbd8c360c96857a5ecc1b596bf1630768789bfa45e06eb82796a0), plus this package's entry and release notes in the published v2/v3 lanes and the legacy alias. No other package or catalog entry changes; the preview overlay is untouched.Package and security impact
relationship-trackerclient.jschanges, and it makes no new requests. The only new storage is the one localStorage key for the collapsed state.Validation
node scripts/validate-catalog.mjspasses locallygit diff --checkpasses locallyCONTRIBUTING.mdManual verification notes
npm run check(0 errors; its 809 warnings are already on staging and none are in this package),node scripts/test-catalog-lanes.mjs,node scripts/validate-package-locales.mjs,node scripts/validate-catalog.mjs,node scripts/tests/catalog-release-notes.regression.mjs, andgit diff --checkall pass. A second run ofnode scripts/build-relationship-tracker-package.mjsproduces no changes.Documentation impact
UI evidence (if applicable)
Screenshots from the local harness attached: before vs after in the Tracker Panel, the collapsed row beside the built-in trackers, and the Trackers window with the new toolbar tile.
🤖 Generated with Claude Code
Summary by CodeRabbit