Skip to content

Trust: north-star5-fast red — spine missing FilterFloatJitHelper + StringFilterFloat after #29028 #29035

Closed

Description

@PurHur

Category

bug · Pillar 1 trust · spine coverage gate (#1945 / #1922)

Problem

make north-star5-verify-fast fails on tip after #29028 (FILTER_VALIDATE_FLOAT / ALLOW_THOUSAND) landed two new vm.php-path units without adding them to compiler_lib_spine_smoke. Release-readiness also fails the spine-coverage gate (and reports 9 stale ext/filter helper-runtime objects — related but separate refresh).

Gate Result Symptom
php script/bootstrap-inventory.php --check green* OK 7252/7252 (*doc may need regen in some worktrees)
php script/check-selfhost-spine-coverage-sync.php red missing ext/filter/FilterFloatJitHelper.php, lib/JIT/Builtin/StringFilterFloat.php
make north-star5-verify-fast red step 2 FAILED (7248/7252)
php script/check-helper-runtime-prelink.php --strict red 9 stale ext/filter/* units — refresh via emit-helper-runtime-object.php --prelink

Introduced by: 4474db1a64 — Stdlib: FILTER_VALIDATE_FLOAT honors ALLOW_THOUSAND (#29013) (#29028).

php-src reference

N/A (trust/gate honesty — not a Zend behavior gap). Sibling of #1945 / #1922 spine coverage guards.

PHP implementation target

  • test/selfhost/compiler_lib_spine_smoke/main.php — add require_once for the two missing inventory files (via php script/bootstrap-selfhost-next-includes.php --bundle=test/selfhost/compiler_lib_spine_smoke/main.php --limit=2)
  • Optionally refresh helper-runtime prelink for the 9 stale filter units (honest rebuild, no restamp)
  • No new C

Repro

./script/docker-exec.sh -- bash -lc 'php script/check-selfhost-spine-coverage-sync.php; make north-star5-verify-fast'

Done when

  • php script/check-selfhost-spine-coverage-sync.php exits 0 (7252/7252 or current inventory)
  • make north-star5-verify-fast green (or first failure is no longer spine coverage)
  • No fingerprint restamp without a build receipt (artifact-honesty)
  • Prefer also clearing check-helper-runtime-prelink.php --strict for the 9 stale filter units when feasible in the same PR

Activity

  1. added
    bugSomething isn't working
    implementation-readySpec complete: repro, php-src ref, done-when — safe for workers to claim
    on Aug 8, 2026
  2. PurHur commented on Aug 8, 2026

    @PurHur
    OwnerAuthor

    Stocked by Worker lane A after mandatory try-first gates. Issue number #29035 % 3 == 1 → Lane B — not claiming (lane discipline).

    First failing command: php script/check-selfhost-spine-coverage-sync.php (also make north-star5-verify-fast step 2).

  3. PurHur commented on Aug 8, 2026

    @PurHur
    OwnerAuthor

    claim: Worker lane B — starting this run (Pillar 1 trust / spine coverage after #29028)

  4. PurHur commented on Aug 8, 2026

    @PurHur
    OwnerAuthor

    Merged #29039

    Gates (before claim): inventory OK 7252/7252; spine-coverage + north-star5-fast red (missing FilterFloat units after #29028).

    Fix: require_once for FilterFloatJitHelper + StringFilterFloat in spine smoke; inventory regen; footnotes → 7250/7252.

    Verified: coverage sync OK, spine-count 7250/7252, deferred sync OK, north-star5-verify-fast OK. No gen-0 restamp. Helper-runtime filter refresh not in this PR (stale ≠ incorrect).

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    area:toolingTooling / CI / docsbugSomething isn't workingimplementation-readySpec complete: repro, php-src ref, done-when — safe for workers to claimphase-0:FoundationPhase 0 – foundation & DevEx

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions