Skip to content

MBTI-INDEX-52: fix streamed preflight namespace isolation - #3342

Merged
fermatmind merged 1 commit into
mainfrom
codex/mbti-index52-preflight-namespace-fix
Jul 27, 2026
Merged

MBTI-INDEX-52: fix streamed preflight namespace isolation#3342
fermatmind merged 1 commit into
mainfrom
codex/mbti-index52-preflight-namespace-fix

Conversation

@fermatmind

Copy link
Copy Markdown
Owner

What changed

  • Corrects the PHP namespace replacement literals used to isolate the streamed INDEX-52 package and service classes.
  • Adds an executable regression test proving both source namespaces are replaced exactly once.

Why

  • Read-only production preflight run 30267578476/1 failed locally while building the streamed runner because the workflow searched for two literal backslashes instead of one.
  • The failure occurred before the production SSH read step; no production action or write occurred.

Validation

  • cd backend && php artisan test tests/Sre/MbtiIndex52ProjectionRepairPreflightWorkflowTest.php
  • cd backend && ./vendor/bin/pint tests/Sre/MbtiIndex52ProjectionRepairPreflightWorkflowTest.php
  • actionlint .github/workflows/mbti-index52-projection-repair-preflight.yml
  • Exact workflow-equivalent streamed runner generation followed by php -l
  • git diff --check

Deferred items

  • Production preflight rerun remains deferred until this focused repair is merged into latest main.
  • No CMS/database write, publication/indexability mutation, sitemap/llms submission, search submission, deploy, restart, or remote file write is included.

Repository rule impact

  • No authority boundary changes. This only repairs the existing read-only backend control-plane runner.

@fermatmind

Copy link
Copy Markdown
Owner Author

@codex review exact head af712e8

@chatgpt-codex-connector

Copy link
Copy Markdown

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

Reviewed commit: af712e8e4b

ℹ️ 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".

@fermatmind
fermatmind merged commit e9166c5 into main Jul 27, 2026
9 checks passed
@fermatmind
fermatmind deleted the codex/mbti-index52-preflight-namespace-fix branch July 27, 2026 13: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.

1 participant