Skip to content

M1-02: Define PreflightReport v0.1 runtime schema #5

Closed

Description

@pillowtalk-Qy

Fallback owner: Qy

协作与发布:
Implementation may begin immediately.
Merge order is controlled by Qy through the Merge Gate.
If a prerequisite has not merged, keep the PR as Draft or rebase
before final review.

范围:

  • Zod EVM 地址、native 资产、最小单位金额和 Protocol 基础 Schema。
  • Intent、Quote、Selection、Capability、Simulation、Alignment、Decision、Limitations 的完整 PreflightReport v0.1 Schema。
  • TypeScript 类型导出。
  • 关键跨字段约束。

验收:

  • 合法报告可解析。
  • 非法地址、浮点金额、空协议、非法滑点被拒绝。
  • 选中协议必须在 allowedProtocols 中且有成功 Quote。
  • MANUAL_REVIEW 不能与失败模拟或非 PASS 的关键 Alignment 共存。
  • 本 Issue 自带 Vitest Schema 测试。
  • 不引入真实链上数据或地址。

Activity

  1. 6 remaining items

  2. added and removed
    status:blockedBlocked by an unresolved dependency
    on Jul 25, 2026
  3. pillowtalk-Qy commented on Jul 25, 2026

    @pillowtalk-Qy
    ContributorAuthor

    Implementation Authorized

    Maintainer determination: implementation of #5 is now formally authorized.

    Decision and gate evidence:

    ADR 0001 and ADR 0002 together are the fixed PreflightReport v0.1 contract. The exact asset, availability, selection, simulation, provenance, ProtocolId, EIP-55, reportId, generatedAt, network, and source-reference decisions must be implemented as recorded; they are not implementation-PR discretion.

    Execution owner: @pillowtalk-Qy, assigned under the Issue fallback-owner rule. Issue status is Open with status:ready, and Project Status is Ready.

    Authorization scope and boundaries:

    Audit note: GitHub temporarily changed the Issue state after interpreting a negated phrase in PR #14 as a closing keyword. The PR body now has no closing reference, and #5 was reopened before this authorization; that transient state was not an implementation completion event.

  4. added
    status:blockedBlocked by an unresolved dependency
    and removed on Jul 26, 2026
  5. pillowtalk-Qy commented on Jul 26, 2026

    @pillowtalk-Qy
    ContributorAuthor

    Maintainer implementation block

    Issue #5 is blocked on the trust-boundary fixes required by the Maintainer review of Draft PR #15 at reviewed head 70c6cf4ee63bbd42d0eaf2a19474436edfc22cf5.

    Assignee, M1 Milestone, and the Issue/PR relationship remain unchanged. Implementation may resume only after PR #15 produces a corrected head and passes the required full Maintainer re-review.

  6. removed their assignment
    on Jul 27, 2026
  7. pillowtalk-Qy commented on Jul 27, 2026

    @pillowtalk-Qy
    ContributorAuthor

    Maintainer repair-owner handoff

    Issue #5 remains Open / Blocked, with Project status Blocked, pending repair of PR #15. The previous fallback implementation assignment does not constitute explicit acceptance of this newly defined trust-boundary repair package, so it has been cleared to avoid showing an unconfirmed repair owner.

    Repair owner: currently unassigned / awaiting explicit acceptance. No completion date is being set while the work is unowned.

    A contributor may claim the repair by commenting on this issue that they accept the complete scope below. The Maintainer will assign the issue only after that explicit acceptance:

    1. Make the public @moss-mini-demo/report-schema runtime entry loadable under supported Node 22 and add the package-import smoke test to the quality gate.
    2. Reject AVAILABLE critical evidence whose raw artifact is null, absent, or otherwise unprovable.
    3. Reject SourceReference self-reference, cycles, metadata self-authentication, unrelated references, and dangling references.
    4. Stop applying presentation/prose/extension path restrictions to identically named fields inside legitimate raw payloads.
    5. Separate public JSON Pointer syntax validation from full-report contextual SourceReference validation.
    6. Add regressions for nullable raw evidence, cycles, and the complete STOP-trigger association matrix, including correct links that pass and unrelated/dangling/cyclic links that fail.

    The repair must be submitted as a genuine new PR #15 head. Green CI on the unchanged head is not evidence that these blockers are resolved. PR #15 stays Draft and Blocked until the new head satisfies the recorded re-entry conditions and completes a fresh Maintainer review.

    Control-plane correction: #15 (comment)

  8. pillowtalk-Qy commented on Jul 27, 2026

    @pillowtalk-Qy
    ContributorAuthor

    Trust-boundary repair scope acceptance

    I accept the complete PR #15 trust-boundary repair scope recorded in the Maintainer repair-owner handoff, including package entry, non-null raw artifacts, contextual SourceReference validation, raw-path ownership, public API naming, and the complete regression matrix.

    I will keep the repository read-only and will not switch to or modify the PR #15 branch until the Maintainer explicitly reassigns Issue #5 to Qy.

  9. pillowtalk-Qy commented on Jul 27, 2026

    @pillowtalk-Qy
    ContributorAuthor

    Maintainer implementation authorization: PR #15 trust-boundary repair

    Qy's complete-scope acceptance is accepted. Issue #5 is now formally assigned to pillowtalk-Qy, and its Project status is In Progress.

    Authorization is granted to update the existing codex/issue-5-preflight-report-schema branch and deliver the repair through the existing PR #15. At the time of authorization, PR #15 remains at 4d47ead0e66b9748dd69e134c214cb0be8c9e480.

    Execution constraints

    Required repair scope

    1. Node 22 can load @moss-mini-demo/report-schema through its public package entry.
    2. The public package-import smoke test is enforced by quality-gate.
    3. AVAILABLE critical raw artifacts reject null, missing, and otherwise unprovable values.
    4. SourceReference validation rejects self-reference, cycles, metadata self-authentication, dangling references, and unrelated references.
    5. Legitimate raw payload fields are not rejected by presentation-layer field-name restrictions.
    6. The public API clearly distinguishes JSON Pointer syntax validation from full-report contextual SourceReference validation.
    7. Regression coverage includes nullable raw artifacts, cyclic references, and the complete STOP-trigger association matrix, including valid associated references that pass and unrelated, dangling, or cyclic references that fail.

    This authorization makes Issue #5 executable; it is not Review or Merge authorization for the current PR head. No delivery date is inferred from scope acceptance because no credible estimate has been supplied.

    Maintainer scope record: #5 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions