Skip to content

refactor(internal-explorer): read bundle and rejected views from audit RPC only - #176

Merged
niran merged 1 commit into
mainfrom
toshi/migrate-explorer-rpcs
Sep 30, 2026
Merged

niran merged 1 commit into
mainfrom
toshi/migrate-explorer-rpcs

Conversation

@niran

@niran niran commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Part of CHAIN-5418: stop reading the base-bundle-history-* S3 buckets from Internal Explorer so they can be deleted.

  • Bundle and rejected views read only the audit events RPC (Postgres-backed). The S3 fallback is gone. When audit is not configured for a chain, bundle returns 404 and rejected returns an empty list. When the audit RPC errors, both return 502 instead of quietly falling back.
  • S3 block cache dropped. The block route already rebuilds from the execution RPC on every request, so the cache only served block hashes the RPC could not find. Those lookups now 404. This removes the explorer's only S3 write (PutObject).
  • Block view no longer fills in bundle id and metering from S3 when audit has no events for a tx.
  • Transaction view drops the S3 archive source: the lookup no longer reads transactions/by_hash or bundles/, the response no longer has archive / coverage.archive, and the page no longer shows the Archive badge or archive wording.
  • Deletes app/api/internal-explorer/s3.ts, the S3 client/config in config.ts, the TIPS_<CHAIN>_S3_* entries in .env.example, and the @aws-sdk/client-s3 dependency.

After merge, protocols-ui will bump its base/ui pin and remove the ui_tips_s3 IAM policies and TIPS_*_S3_* chart env.

Screenshots

Transaction page (/internal-explorer/txn/[hash], mainnet tx, audit not configured locally): the Archive provenance badge is gone and the copy now says Audit.

internal-explorer-txn

Internal Explorer home (unchanged layout, for reference):

internal-explorer

Test plan

  • npm test: 55 files, 408 tests pass (transaction-lookup tests updated for the audit-only lookup).
  • npx eslint app/api/internal-explorer app/internal-explorer: no new findings.
  • npm run build:internal succeeds.
  • tsc --noEmit has one error, a missing PNG type declaration in app/vibenet/demos/200/BlockRunner.tsx, which this PR does not touch.

Generated with Toshi

type=routine
risk=low
impact=sev5

…t RPC only

Drop the S3 archive fallback and the S3 block cache (CHAIN-5418). Bundle,
rejected, block, and txn views now read only the Postgres-backed audit RPC
and the execution RPC. Removes @aws-sdk/client-s3 and TIPS_<CHAIN>_S3_* env.
@cb-heimdall

cb-heimdall commented Sep 29, 2026 •

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews ✅ 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@vercel

vercel Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
omni-ui Ready Ready Preview Sep 29, 2026 7:20pm UTC

Request Review

@linear

linear Bot commented Sep 29, 2026

Copy link
Copy Markdown

CHAIN-5418

@niran
niran added this pull request to the merge queue Sep 30, 2026
Merged via the queue into main with commit 6caa8f4 Sep 30, 2026
13 checks passed
@niran
niran deleted the toshi/migrate-explorer-rpcs branch September 30, 2026 00:51

This branch was successfully deployed

1 active deployment
Preview — d458265d Deployed Sep 29, 2026 by vercel[bot]
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.

3 participants