Repository navigation
Show source-backed values with readable views and explicit inspection - #38
Open
rororowyourboat wants to merge 5 commits into
Open
rororowyourboat wants to merge 5 commits into
rororowyourboat wants to merge 5 commits into
Conversation
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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.
Change
Configuration and evidence nodes previously lost authored values between the native renderer and browser projection. This change carries exact source expressions, providers and occurrence contexts through the validated worker contract and displays bounded previews with full Inspector details. Keyword-free redefinitions and distinct nested usage overrides are covered; unsupported binding/precedence paths are explained without inventing evaluated values.
Large authored views now enter at readable scale when a complete fit would fall below 60%. Fit view provides the overview and 100% focuses the inspected occurrence or port owner. Tree/table selection opens details, with source navigation as an explicit action guarded against changed drafts. Saved geometry, portable data and engine pins stay unchanged.
Fixes #24. Addresses the readability and inspection workflows in #30, #31 and #33. The structural column layout remains unchanged after bounded layout probes; general layout repacking and broader navigation feedback remain follow-up work, so those issues are not automatically closed.
Evidence
npm run check: formatting, architecture, TypeScript, 179 unit tests and 10 script tests passed.Full functional, Firefox/WebKit and comprehensive accessibility suites were not run. Local results do not establish hosted CI or deployment.
Compatibility
Derived worker metadata is additive and validated against applied source. No dependency, engine pin or saved-format change. No expression evaluation, generalized binding precedence, library source navigation or deployment is claimed. See docs/verification.md for exact pins, negative controls, review boundaries and retained evidence.