Skip to content

Let the inventory dialog navigate the project hierarchy - #20

Closed
EiffL wants to merge 1 commit into
mainfrom
feat/inventory-hierarchy
Closed

EiffL wants to merge 1 commit into
mainfrom
feat/inventory-hierarchy

Conversation

@EiffL

@EiffL EiffL commented Sep 17, 2026

Copy link
Copy Markdown
Member

Summary

  • @astra-spec/ui's Inventory now renders the project hierarchy in its sidebar and calls onSelectAnalysis when a controlled host's selection should change (Inventory: simplify sections and add analysis navigation astra-ui#27). The inventory dialog now owns that selection: it opens on the page's analysis, its title follows whichever analysis is selected, and it is keyed by the page's analysis so a page change or reopening starts there again.
  • The existing publication inventory tests still pass, including the page-change case.

Before merging

  • @astra-spec/ui is pinned to the pkg.pr.new review build of astra-ui 45416e7; switch it to the released version once astra-ui#27 ships. (npm 12 refuses remote tarballs by default; locally this needed npm install --allow-remote=all.)

npm run typecheck and npm test pass locally.

🤖 Generated with Claude Code

astra-ui's Inventory now shows the project hierarchy in its sidebar and
needs a host callback to act on a controlled selection. The dialog owns
that selection: it starts from the page's analysis, its title follows the
selected one, and it is keyed by the page's analysis so a page change or
reopening starts there again.

The dependency points at astra-ui PR #27's review build until that change
is released.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

Lightcone previews: Paper · Playground — inventory, viewers, dialogs and glyphs

d2a9678 · @astra-spec/ui 0.0.5 from 9c940eb · Lightcone brand 0.0.3 · astra-theme d2a9678 · content 3735fcc · MySTRA https://github.com/LightconeResearch/MySTRA/releases/download/v0.0.8/mystra.mjs

@EiffL EiffL closed this Sep 17, 2026
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