Skip to content

From #775's review: the popup's type lists releaseStatus; ABANDONED pinned; comments - #776

Merged
adamjohnwright merged 1 commit into
mainfrom
fix/775-review-notes
Nov 2, 2026
Merged

adamjohnwright merged 1 commit into
mainfrom
fix/775-review-notes

Conversation

@adamjohnwright

Copy link
Copy Markdown
Contributor

No spec: follow-up to #775's review (all low).

  • The entity popup's lower-pathways answer carries releaseStatus in Release 98. The popup's type lists it, the call table follows, and the route's known problems go down by that one (1060 → 1059).
  • check-har's ABANDONED entry for the legacy-link tests names the pathways they open (1280218 and 202733, by dbId or stable id) rather than any id. The analysis-results entry no longer matches anything recorded, so it is removed.
  • Two comments said Circadian clock carries foreignObjects. Release 98's copy has none; the check stays for artwork exported that way again.

Not done: a recorded deleted search entry to restore that kind's coverage. The search falls back to deleted objects only when nothing live matches, and no query I could find through the public API returns one on Release 98.

🤖 Generated with Claude Code

…inned; comments

- The entity popup's lower-pathways answer carries releaseStatus in
  Release 98 (a pathway new or updated says which, as Event has it). The
  popup's type lists it, the call table follows, and the route's known
  problems go down by that one (1060 to 1059).
- check-har's ABANDONED entry for the legacy-link tests names the pathways
  they open (1280218 and 202733, by dbId or stable id) rather than any id;
  the analysis-results entry no longer matches anything recorded and goes.
- Two comments said Circadian clock carries foreignObjects; Release 98's
  copy has none, and the check stays for artwork exported that way again.

Not done: a recorded deleted search entry to restore that kind's coverage.
The search falls back to deleted objects only when nothing live matches,
and no query found through the public API returns one on Release 98.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@adamjohnwright
adamjohnwright merged commit b2181ee into main Nov 2, 2026
8 checks passed
@adamjohnwright
adamjohnwright deleted the fix/775-review-notes branch November 2, 2026 16:46
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