Skip to content

feat(web): add offline manifest inspection tooling - #16

Merged
Arvolear merged 1 commit into
devfrom
feature/tx-manifest-tooling
Sep 14, 2026
Merged

Arvolear merged 1 commit into
devfrom
feature/tx-manifest-tooling

Conversation

@lukachi

@lukachi lukachi commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add a public, read-only transaction-manifest inspection API and runtime registry reporting.
  • Add offline Manifest Inspector and Format Support pages to the web app.
  • Extend the temporary stacked CI gate with package, smplx-adapter, and tooling typechecks, a direct Vite bundle, formatting, lint, and focused tests.

Stack

PR 7 of 9. This PR targets PR 6 (#15), feature/tx-manifest-validation.

Boundary

This slice inspects documents offline only. It does not connect a wallet or network, authorize or sign transactions, finalize or broadcast them, or activate release behavior. The committed smplx submodule pointer is unchanged.

The inspector follows the PR 6 model: foreign-asset remains unreachable without request/deployment context, unreadable-build-mode is document-only, and compiler version is an explicit optional input because the current package does not expose it.

Verification

Verified from a detached worktree with the exact committed LWK and smplx revisions:

  • package typecheck
  • focused manifest-tooling typecheck
  • exact smplx adapter typecheck
  • direct Vite production bundle
  • formatting check
  • lint (existing repository warnings only)
  • 653 tests across 36 files, 0 failures

Known deferred gate

The full root/web dependency typecheck still exposes the existing duplicate React/Zod/csstype dependency-family debt tracked as F002 and reserved for U009. This PR adds a focused tooling typecheck so its own source and imported UI dependencies are checked without claiming that broader debt is solved.

A local browser visual smoke test could not run because the connected browser runtime had no available browser. Server-render tests and the production bundle passed.

Base automatically changed from feature/tx-manifest-validation to dev September 14, 2026 15:59

@Arvolear Arvolear left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM as it's mostly a web app.

@Arvolear
Arvolear merged commit e44e30b into dev Sep 14, 2026
2 checks passed
@Arvolear
Arvolear deleted the feature/tx-manifest-tooling branch September 14, 2026 16:20
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.

2 participants