Skip to content

Bootstrap: LanguageProfileScope spine coverage + #17681 test guards - #17689

Merged
PurHur merged 1 commit into
masterfrom
agent/issue-17681-spine-language-profile-scope
Jul 10, 2026
Merged

PurHur merged 1 commit into
masterfrom
agent/issue-17681-spine-language-profile-scope

Conversation

@PurHur

@PurHur PurHur commented Jul 10, 2026

Copy link
Copy Markdown
Owner

Summary

Root cause

PR #17685 introduced LanguageProfileScope on the bin/vm.php path but did not add the spine require_once, leaving spine coverage sync red.

Verification

php script/check-selfhost-spine-coverage-sync.php
# check-selfhost-spine-coverage-sync: OK (spine covers all 4338 inventory files)

php script/bootstrap-inventory.php --check
# OK 4338/4338

vendor/bin/phpunit --filter LanguageProfileScopeTest
# OK (5 tests, 15 assertions)

vendor/bin/phpunit --filter exit_die_named_status
# OK (6 tests, 6 assertions)

php bin/vm.php test/repro/maintainer_gap_exit_named_status.php  # exit 0
php bin/vm.php test/repro/maintainer_gap_die_named_status.php   # exit 1

Follow-up to Closes #17681 (main fix merged in #17685).

Made with Cursor

PR #17685 merged LanguageProfileScope but omitted the self-host spine
require; restore coverage sync and add die/pragma compliance + manifest tests.

Co-authored-by: Cursor <cursoragent@cursor.com>
@PurHur
PurHur merged commit d801748 into master Jul 10, 2026
1 check failed
@PurHur
PurHur deleted the agent/issue-17681-spine-language-profile-scope branch July 10, 2026 19:00
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.

Language: exit(status:)/die(status:) named parameters — parse error without PHP_COMPILER_PROFILE=8.4 (Zend/zend_compile.c)

1 participant