Skip to content

Publish Calibration direct-read dashboard evidence - #38

Merged
snissn merged 1 commit into
mainfrom
codex/issue-33-calibration-evidence
Jul 2, 2026
Merged

Publish Calibration direct-read dashboard evidence#38
snissn merged 1 commit into
mainfrom
codex/issue-33-calibration-evidence

Conversation

@snissn

@snissn snissn commented Jul 2, 2026

Copy link
Copy Markdown
Member

Summary

Closes #33.

This publishes the direct-onchain Calibration evidence for the admin dashboard stack under #27/#29/#30/#31/#32:

  • refreshes artifacts/calibration/demo-evidence.json against the pagination-capable Calibration FocPlatformRegistry at 0x8F6563Bb9E53aeDfE9d87d4C1E162f0371649c18
  • updates Worker public vars and generated Wrangler env types so /api/admin/* defaults to live direct reads with a runtime hash matching the current registry artifact
  • extends the Calibration runner to record dashboard count/list/detail/readBatch proof and to allowlist a demo relayer row
  • updates README/deployment/API/Worker docs from pending Publish Calibration evidence for direct-onchain dashboard reads #33 language to current direct-read evidence
  • deploys the public Worker demo at https://foc-platform-calibration-demo.snissn.workers.dev with version c83bd91f-164e-4a48-9d0a-06518cd51212

Public Calibration Evidence

Registry:

  • address: 0x8F6563Bb9E53aeDfE9d87d4C1E162f0371649c18
  • chain: Filecoin Calibration 314159
  • deploy tx: 0xae42c13c50c1b268a1d38389e27d8fa776264b405e28a1cf11a974dd4b178eae
  • deploy block: 3854411
  • runtime SHA-256: 0x2c49443e7a9ebf3337453240e706df249d29f4f217ec948d6c10e9502a199d1f

Demo object:

  • object id: 1
  • account id: 0xfaa4a252e3d762275f3bb2baccba097024ed47a08845bee0de79a2ee58514e01
  • provider/dataset/piece: 4 / 12524 / 34
  • status: Committed
  • dashboard proof in artifact: objectCount=1, accountCount=1, datasetRecordCount=1, coordinatorCount=1, relayerCount=1, readBatch.resultCount=5

Registry write txs:

  • setCoordinator: 0x22a29586247fa39e8d3277c754a42466f1936a3377f2d62ab50ddf1337035cd1
  • setRelayer: 0x93153de2f1653b1e632dff19f2dfd632f5f2a6390af20b39097a53607c162570
  • requestUpload: 0x35c561f8b6278b63deb1d272b8cc2b7663a45eb3cc564e75092bbeb5ebf12ccf
  • startUpload: 0x6c57ded5248162049d64d372c2d1bde7e37c788edeedafc454bbbd2e3f841f38
  • recordDataset: 0xb65836f00e8b7a24734e3d5e62d64ad305e3234fd879e2a435b1665d273b6669
  • finalizeUpload: 0xe046f47af71e7da40ed475aba0e3ba8ac94677ae2de7341e1b59e2f8e367a2a4

Validation

  • node --test test/calibration-worker.test.mjs
  • pnpm lint
  • pnpm ops:validate
  • pnpm ops:smoke -- --iterations 3
  • pnpm worker:dry-run
  • pnpm test
  • npx wrangler deploy
  • Live Worker checks:
    • curl https://foc-platform-calibration-demo.snissn.workers.dev/api/health
    • curl https://foc-platform-calibration-demo.snissn.workers.dev/api/admin/overview
    • curl 'https://foc-platform-calibration-demo.snissn.workers.dev/api/admin/files?limit=10'
    • curl 'https://foc-platform-calibration-demo.snissn.workers.dev/api/admin/coordinators?limit=10'
    • curl https://foc-platform-calibration-demo.snissn.workers.dev/api/demo/registry

Boundaries

  • No private key, wallet seed, session key, or privileged signer material is committed or exposed to the Worker.
  • The Worker remains read-only: it serves public evidence and direct registry reads only.
  • This closes direct dashboard read proof, but production session-key coordinator, expiry/revoke, real payment, and KMS paths remain compatibility-gated outside this PR.

Copilot AI review requested due to automatic review settings July 2, 2026 02:37

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Filecoin Calibration “direct onchain-read” demo to point at a new pagination-capable FocPlatformRegistry deployment and publishes refreshed public evidence (artifact + docs) showing the admin dashboard can be driven by bounded count/list/detail/readBatch contract views.

Changes:

  • Refreshes the committed demo registry address / deploy evidence / runtime SHA and updates Wrangler + generated Worker env types accordingly.
  • Extends the Calibration registry demo runner to (a) allowlist a demo relayer row and (b) record dashboard direct-read proof into the committed evidence artifact.
  • Updates README and docs to reflect that dashboard APIs now default to live direct reads when the runtime hash matches the current artifact.

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
wrangler.jsonc Updates committed Worker vars to the new Calibration registry + runtime hash and tx hash evidence.
test/calibration-worker.test.mjs Adds coverage for runtime-hash-gated live dashboard defaults using a “stale” runtime SHA.
src/worker/worker-configuration.d.ts Regenerates Wrangler env typings to match the updated committed vars.
src/worker/calibration-demo.mjs Updates default registry constants and sets default evidence mode to calibration_live_evidence.
scripts/run-calibration-registry-demo.mjs Records direct dashboard count/list/detail/readBatch proof and allowlists a demo relayer.
README.md Updates narrative and demo guidance to reflect default live direct reads when ABI matches.
docs/platform-api.md Clarifies current admin rows should come from direct contract list/count reads (events remain audit/fixture).
docs/deployment.md Adds pointer to the current Worker demo evidence artifact and what it proves.
docs/calibration-worker-demo.md Updates the demo runbook and published evidence values for the new registry.
artifacts/calibration/demo-evidence.json Refreshes the committed public evidence snapshot (including dashboard proof).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/calibration-worker-demo.md

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7017567e5c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/worker/calibration-demo.mjs
@snissn
snissn force-pushed the codex/issue-33-calibration-evidence branch from 7017567 to c372414 Compare July 2, 2026 02:42
@snissn

snissn commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

@codex review current head c372414 after the Copilot documentation fix.

@snissn
snissn force-pushed the codex/issue-33-calibration-evidence branch 2 times, most recently from 7f58b15 to 81df036 Compare July 2, 2026 02:48
@snissn

snissn commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

@codex review current head 81df036 after addressing the runtime-hash gate and CI gas-limit feedback.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Another round soon, please!

Reviewed commit: 81df036c0a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@snissn
snissn force-pushed the codex/issue-33-calibration-evidence branch from 81df036 to 5b08b49 Compare July 2, 2026 02:56
@snissn

snissn commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

@codex review current head 5b08b49 after final runner runtime-hash guard alignment.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5b08b497ed

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/run-calibration-registry-demo.mjs Outdated
@snissn
snissn force-pushed the codex/issue-33-calibration-evidence branch from 5b08b49 to 23af809 Compare July 2, 2026 03:07
@snissn

snissn commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

@codex review current head 23af809 after custom registry deploy metadata guard alignment.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 23af809592

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/worker/calibration-demo.mjs Outdated
@snissn
snissn force-pushed the codex/issue-33-calibration-evidence branch from 23af809 to ba98bf6 Compare July 2, 2026 03:14
@snissn

snissn commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

@codex review current head ba98bf66db93381b26a88551c0ea3ca0f001d81e after Worker deploy metadata guard alignment.

@snissn

snissn commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

@codex review current head ba98bf6 after Worker deploy metadata guard alignment.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep them coming!

Reviewed commit: ba98bf68ea

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@snissn

snissn commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

@codex review current head ba98bf6. Please re-review after the Worker and runner custom registry metadata fixes.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

Reviewed commit: ba98bf68ea

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@snissn

snissn commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Swish!

Reviewed commit: ba98bf68ea

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@snissn

snissn commented Jul 2, 2026

Copy link
Copy Markdown
Member Author

Merge-gate update for final head ba98bf68ea8c82cbae0d7cbf8c8c26e556521da3:

  • Latest-head CI is green: CI / test, run 28562762411, completed 2026-07-02T03:15:19Z.
  • Local gates passed after the final metadata fixes: node --test test/calibration-worker.test.mjs, pnpm lint, pnpm test, and pnpm worker:dry-run.
  • All GitHub review threads are resolved.
  • GitHub Codex review findings from earlier heads were fixed and resolved. Current-head Codex was requested repeatedly (@codex review current head ... and final @codex review) after the mature head, but the GitHub bot did not respond after repeated waits. Treating that reviewer as unavailable/nonresponsive for this final head per the mergeability skill fallback, not as a pass.
  • Independent Codex subagent review of the current head found no merge-blocking issues in the Worker/runner custom registry metadata paths; residual risk was limited to not re-running the full suite in the subagent, which the coordinator already ran locally.

@snissn
snissn merged commit 8964571 into main Jul 2, 2026
1 check passed
@snissn
snissn deleted the codex/issue-33-calibration-evidence branch July 2, 2026 03:37
@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create an environment for this repo.

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.

Publish Calibration evidence for direct-onchain dashboard reads

2 participants