Skip to content

docs: synchronize official baseline 3 after ERC-8350 revision merge - #39

Merged
XiaoHai67890 merged 1 commit into
mainfrom
docs/official-baseline-3-20261106
Nov 6, 2026
Merged

XiaoHai67890 merged 1 commit into
mainfrom
docs/official-baseline-3-20261106

Conversation

@XiaoHai67890

Copy link
Copy Markdown
Collaborator

Summary

  • Synchronize the official Draft mirror to Update ERC-8350: clarify authorization and preserve ERC-1271 fallback ethereum/ERCs#2147 merge 8b11215cf267f710b49ea003384e678bcf953eae.
  • Record spec-baseline-3; preserve baseline-1, baseline-2 and the historical candidate manifest byte-for-byte.
  • Add an explicit official-source evidence mode, cross-bind manifest/spec/assets to baseline 3, retain historical-candidate and old-source regression runs.
  • Align the authorization table, resolved/open questions, audit boundary and Draft-to-Review packet. No product runtime, hash, wire, vector, frozen audit or deployment changes.

Verification

  • 47 baseline/source-gate unit tests PASS; zero skips.
  • Pinned upstream text verification PASS (SHA-256 and Git blob).
  • PR test: pin official authorization candidate and Review evidence #38 merged as 086e493d70bb07fc2e8b4bfac12282312c2cdc9e; its main CI, real-account and official-candidate runs all succeeded.
  • Fresh clean-checkout, official-source and account checks for this PR are being run. Exact commit-bound results will be added below; this is not a claim that pending checks already passed.

Acceptance boundaries

The official status remains Draft. This does not accept G4, update an immutable deployment, or extend independent Python evidence to ERC-1271/Safe/7702. Current-runtime external audit acceptance remains pending. Historical failures and pins remain available.

@XiaoHai67890

Copy link
Copy Markdown
Collaborator Author

Verification completed on f5c36e2 (clean checkout; no runtime/vector changes).

  • Clean-checkout conformance: PASS; 155 workspace TypeScript tests, 120 Solidity tests; core TS, minimal TS and both Solidity asset variants match test-vectors/v1.json byte-for-byte.
  • Baseline and source-runner unit tests: 47/47 PASS; pinned official text verification PASS.
  • Exact merged official source and preserved historical candidate: each 38/38 PASS, zero skips; each old-source negative control reproduces exactly the two expected failures.
  • Real Safe/EIP-7702 suite: 58/58 PASS; actual local type-4 transaction, nonce, stale authorization and Registry transition checks PASS.
  • All 8 PR checks are green: conformance/TS/Solidity, source evidence, real accounts, CodeQL. Logs/reports are retained in the workflow artifacts.

G4 remains explicitly NOT accepted: the full security gate fails for missing canonical external-review.json and records four post-freeze source differences. No audit evidence was fabricated or scope relaxed. Official ERC-8350 remains Draft until a separate status PR is accepted.

@XiaoHai67890

XiaoHai67890 commented Nov 6, 2026 •

Copy link
Copy Markdown
Collaborator Author

Publication and Review-request handoff (2026-11-06)

The publication steps in docs/review/draft-to-review.md are now executed, not merely drafted:

Item Result / public record
Project #38 Merged at 086e493d70bb07fc2e8b4bfac12282312c2cdc9e; main checks passed
Project #39 Merged at f654fe023909ef93e1478e79dbcee7cbd7b05835; baseline 3 pins official 8b11215cf267f710b49ea003384e678bcf953eae
Community update Published Magicians post #36
Independent-client handoff Issue #21 update
External-review handoff Issue #11 update
Separate status request ethereum/ERCs#2148, head 670427caebfc425ed37265633307ad70e47b8e55; exactly one line: status: Draft -> status: Review

All four fresh project-main workflows on f654fe0 passed:

Local pinned-head counts and artifacts remain in the verification record. The historical manifests, vectors and frozen audit target were preserved.

At final verification, the official status PR is OPEN at 670427caebfc425ed37265633307ad70e47b8e55: all nine latest named checks report success, including HTMLProofer in the completed CI run. GitHub reports MERGEABLE but BLOCKED; no review has been submitted on this status PR. Official ERC-8350 therefore remains Draft until the status request is merged. The superseded run was cancelled after correcting project-issue links in the PR body; the code commit did not change.

CI qualification: the upstream Markdown Linter job is green but its changed-file filter only matches root Markdown and EIPS/eip-*.md, so it skips ERCS/erc-8350.md. A supplemental local check used the action's pinned markdownlint-cli2@0.6.0 and the official configuration at the status-PR head:

npm exec --yes --package=markdownlint-cli2@0.6.0 -- markdownlint-cli2-config config/.markdownlint.yaml ERCS/erc-8350.md

Result: markdownlint-cli2-config 0.6.0 / markdownlint 0.27.0; 1 file linted; 0 errors; exit 0. No upstream workflow changes were mixed into the one-line status request. Green checks and this supplemental check are not an Editor approval or a completed status transition.

Remaining external work stays open: independent ERC-1271/Safe/7702 coverage; reviewer-controlled current-runtime addendum and canonical G4 acceptance; deployment/release evidence. The full G4 gate still fails on missing audits/g4/external/external-review.json. No independent results, audit acceptance or production readiness are inferred from maintainer tests or publication.

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