Skip to content

[codex] M4 progress: chain intelligence + deep live matrix evidence#97

Draft
Prekzursil wants to merge 16 commits intomainfrom
codex/m4-progress-deep-matrix-20260303
Draft

[codex] M4 progress: chain intelligence + deep live matrix evidence#97
Prekzursil wants to merge 16 commits intomainfrom
codex/m4-progress-deep-matrix-20260303

Conversation

@Prekzursil
Copy link
Owner

Summary

This PR pushes the current M4 progress snapshot for installed workshop/submod intelligence and deep live evidence orchestration. It is intended as an intermediate integration point so we can review current behavior and plan the next closure slice.

User Impact

Before this change, installed-chain execution produced ambiguous submod-parent behavior and deep live coverage was not captured as a complete matrix artifact for the expanded inventory.

With this change:

  • installed workshop inventory and chain metadata are surfaced across runtime/tooling
  • chain-aware launch orchestration is expanded (single and multi-STEAMMOD chains)
  • helper/mechanic/transplant scaffolding and diagnostics are wired into runtime/tooling
  • deep matrix artifacts are persisted and validated for the full resolved chain set
  • evidence contracts are synchronized in TODO.md and runbook/test docs

Root Cause

The remaining gap was not core architecture, but closure-grade execution wiring and evidence coverage:

  • submod parent dependencies were inconsistently represented in chain metadata
  • deep-chain matrix orchestration needed robust per-chain artifact persistence and blocker visibility
  • newly installed workshop items required inclusion in discovery/seeds/evidence

What Changed

Runtime/Core/Profile surface

  • Added workshop inventory contracts/models and service wiring
  • Added mod mechanic detection contracts/models and service wiring
  • Added transplant contracts/models and SwfocTrainer.Transplant project scaffolding
  • Expanded runtime action/reliability wiring for new M4 operation surfaces
  • Updated base profiles/actions, helper bridge/native bridge pathways, and catalog/runtime integration

Tooling and validation

  • Extended tools/run-live-validation.ps1 for full installed-chain deep execution + matrix output
  • Extended repro bundle generation/schema/validator for M4 evidence fields
  • Added transplant report schema + validator + export tooling
  • Updated generated seed fixtures (including newly installed IDs)

Tests

  • Added/updated tests for:
    • workshop inventory discovery/chain handling
    • launch service
    • runtime context-faction routing
    • mechanic detection
    • helper backend behavior
    • catalog and transplant services

Verification

Deterministic:

  • dotnet build SwfocTrainer.sln -c Release --no-restore
  • dotnet test tests/SwfocTrainer.Tests/SwfocTrainer.Tests.csproj -c Release --no-build --filter "FullyQualifiedName!~SwfocTrainer.Tests.Profiles.Live&FullyQualifiedName!~RuntimeAttachSmokeTests" ✅ (Passed: 270)
  • powershell.exe -File ./tools/validate-workshop-topmods.ps1 -Path tools/fixtures/workshop_topmods_sample.json -Strict
  • powershell.exe -File ./tools/validate-generated-profile-seed.ps1 -Path tools/fixtures/generated_profile_seeds_sample.json -Strict

Live/deep evidence:

  • Deep matrix run: LIVE-M4-DEEP-20260302
    • matrix: TestResults/runs/LIVE-M4-DEEP-20260302/chain-matrix-summary.json
    • resolved chains: 28
    • classifications: 26 skipped, 2 blocked_environment
  • Targeted reruns:
    • LIVE-M4-RERUN-CHAIN16-20260303 => blocked_environment (persistent)
    • LIVE-M4-RERUN-CHAIN27-20260303 => skipped (transient blocker cleared)
  • New installed mod evidence included:
    • LIVE-NEWMOD-1780988753-20260303
    • LIVE-NEWMOD-2361944372-20260303

Reason-Code Diagnostics

  • Persistent blocker: ATTACH_NO_PROCESS on chain 2313576303 (LIVE-M4-DEEP-20260302-chain16, rerun confirms)
  • ROE precondition skip remains explicit: set_credits precondition unmet: hook sync tick not observed on ROE chain runs

Risk and Rollback

  • risk:high
  • Runtime/tooling behavior changed across launch/inventory/mechanic/transplant and evidence contracts.
  • Rollback path: revert this commit to restore prior M3/M4 baseline behavior and tooling contracts.

Evidence References

  • TestResults/runs/LIVE-M4-DEEP-20260302/chain-matrix-summary.json
  • TestResults/runs/LIVE-M4-DEEP-20260302/chain-matrix-summary.md
  • TestResults/runs/LIVE-M4-DEEP-20260302-chain16/repro-bundle.json
  • TestResults/runs/LIVE-M4-DEEP-20260302-chain27/repro-bundle.json
  • TestResults/runs/LIVE-M4-RERUN-CHAIN16-20260303/repro-bundle.json
  • TestResults/runs/LIVE-M4-RERUN-CHAIN27-20260303/repro-bundle.json
  • TestResults/mod-discovery/LIVE-M4-DEEP-20260302/installed-mod-graph.json
  • TestResults/mod-discovery/LIVE-M4-DEEP-20260302/resolved-launch-chains.json

Prekzursil and others added 3 commits March 1, 2026 02:44
Add SWFOC_FORCE_PROMOTED_EXTENDER override semantics in BackendRouter with default non-forced promoted routing, de-trap universal_auto backend preference, remove hardcoded user save-root metadata, and align tests/docs/TODO evidence for deterministic and tactical live validation runs.

Co-authored-by: Codex <noreply@openai.com>
Implement helper bridge contracts/backend wiring, native helper plugin routing, strict tactical mode split, and launch automation in runtime/UI.

Stabilize deterministic/live validation by isolating named-pipe tests, hardening repro bundle parsing, improving ROE auto-launch mod args, and recording latest evidence run IDs in docs/TODO.

Co-authored-by: Codex <noreply@openai.com>
Continue M4 closure with installed workshop/submod intelligence, chain-aware launch orchestration, helper/mechanic gating, transplant scaffolding, and expanded evidence tooling.

Includes full deep-chain live matrix artifacts for 25 installed items / 28 chains, plus targeted blocker reruns and TODO evidence synchronization.

Co-authored-by: Codex <noreply@openai.com>
@devloai
Copy link

devloai bot commented Mar 3, 2026

Unable to trigger custom agent "Code Reviewer". You have run out of credits 😔
Please upgrade your plan or buy additional credits from the subscription page.

@coderabbitai
Copy link

coderabbitai bot commented Mar 3, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch codex/m4-progress-deep-matrix-20260303

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Prekzursil Prekzursil added the risk:high High-risk change; strict human gate required. label Mar 3, 2026
Prekzursil and others added 5 commits March 3, 2026 04:45
Reduce complexity/literal duplication in helper bridge, workshop inventory, runtime routing, and related tests without changing runtime semantics.

Co-authored-by: Codex <noreply@openai.com>
Recompute/validate launch chains from items, preserve graph missing-parent diagnostics, and emit chainResolutionSource + launchAttempted metadata so unresolved chains fail closed before attach.

Co-authored-by: Codex <noreply@openai.com>
Wire coverlet collector, deterministic coverage collection/assert scripts, strict codecov.yml policy, and CI workflow for PR/main coverage enforcement.

Co-authored-by: Codex <noreply@openai.com>
Introduce provider-backed zero-issue assertions for Sonar/Codacy/DeepScan/Snyk/Applitools/Sentry with hard-fail missing config policy, plus PR/main workflows and Applitools gate integration.

Co-authored-by: Codex <noreply@openai.com>
Finalize the follow-up Sonar cleanup by removing unused parameters, reducing code smells in workshop inventory URI/path handling, and scoping workflow permissions at job level for visual audit checks.

Co-authored-by: Codex <noreply@openai.com>
Prekzursil and others added 4 commits March 3, 2026 05:15
Fix remaining Codacy complexity/NLOC findings, resolve Sonar workflow hotspots in Codecov analytics, and update auto-launch function semantics to satisfy script analyzer rules while preserving behavior.

Co-authored-by: Codex <noreply@openai.com>
Move chain-resolution helpers into a partial class file so WorkshopInventoryService.cs stays under file-size/NLOC limits without changing runtime behavior.

Co-authored-by: Codex <noreply@openai.com>
Set explicit job display names for Codecov, Snyk, Sentry, and Applitools workflows so branch protection required checks map to the exact expected contexts.

Co-authored-by: Codex <noreply@openai.com>
Consume the cancellation token parameter in the test stub health overload to remove the final Codacy unused-parameter warning.

Co-authored-by: Codex <noreply@openai.com>
@Prekzursil
Copy link
Owner Author

Strict-zero recovery update (Codex)

  • Landed follow-up commits for PR [codex] M4 progress: chain intelligence + deep live matrix evidence #97:

    • e8f8a34 chain16 unresolved-parent classification fix (blocked_dependency_missing_parent pre-launch)
    • 4082aa2 Codecov 100/100 infrastructure
    • f795cfd provider zero-backlog scripts/workflows
    • 739626b, a8bfcb4, 7f16d1b, a9cf06c Sonar/Codacy remediation wave
    • 3cf8961 workflow job-name alignment for required strict-zero contexts
  • Verification snapshots:

    • dotnet build SwfocTrainer.sln -c Release --no-restore passed
    • dotnet test tests/SwfocTrainer.Tests/SwfocTrainer.Tests.csproj -c Release --no-build --filter FullyQualifiedName!~SwfocTrainer.Tests.Profiles.Live&FullyQualifiedName!~RuntimeAttachSmokeTests passed (270 tests)
    • SonarCloud + SonarCloud Code Analysis pass on latest analyzed heads
  • Remaining blockers for full strict-zero closure:

    1. Coverage hard gate still fails at baseline line=44.13, branch=35.69 against required 100/100.
    2. Provider zero scripts hard-fail in strict mode until required credentials/vars are configured (SONAR, CODACY, DEEPSCAN, SNYK, APPLITOOLS, SENTRY).
    3. Codacy provider cycle for the latest head is still reconciling in PR checks.
  • Branch protection on main has been updated to require the expanded strict-zero contexts (Codecov Analytics, Snyk Zero, Applitools Visual Zero, Sentry Zero, DeepScan, plus existing required checks).

Resolve Codacy partial-style notices by moving workshop chain logic into an internal static helper class while keeping WorkshopInventoryService focused and below file NLOC thresholds.

Co-authored-by: Codex <noreply@openai.com>
@Prekzursil
Copy link
Owner Author

Strict-zero recovery update (Codex):\n\n- Landed follow-up commits for PR #97:\n - chain16 unresolved-parent classification fix ( pre-launch)\n - Codecov 100/100 infrastructure\n - provider zero-backlog scripts/workflows\n - , , , Sonar/Codacy remediation wave\n - workflow job-name alignment for required strict-zero contexts\n\n- Current verification snapshots:\n - MSBuild version 17.8.3+195e7f5a3 for .NET
SwfocTrainer.Meg -> C:\Users\Prekzursil\Downloads\SWFOC editor\src\SwfocTrainer.Meg\bin\Release\net8.0\SwfocTrainer.Meg.dll
SwfocTrainer.DataIndex -> C:\Users\Prekzursil\Downloads\SWFOC editor\src\SwfocTrainer.DataIndex\bin\Release\net8.0\SwfocTrainer.DataIndex.dll
SwfocTrainer.Flow -> C:\Users\Prekzursil\Downloads\SWFOC editor\src\SwfocTrainer.Flow\bin\Release\net8.0\SwfocTrainer.Flow.dll
SwfocTrainer.Core -> C:\Users\Prekzursil\Downloads\SWFOC editor\src\SwfocTrainer.Core\bin\Release\net8.0\SwfocTrainer.Core.dll
SwfocTrainer.Helper -> C:\Users\Prekzursil\Downloads\SWFOC editor\src\SwfocTrainer.Helper\bin\Release\net8.0\SwfocTrainer.Helper.dll
SwfocTrainer.Transplant -> C:\Users\Prekzursil\Downloads\SWFOC editor\src\SwfocTrainer.Transplant\bin\Release\net8.0\SwfocTrainer.Transplant.dll
SwfocTrainer.Catalog -> C:\Users\Prekzursil\Downloads\SWFOC editor\src\SwfocTrainer.Catalog\bin\Release\net8.0\SwfocTrainer.Catalog.dll
SwfocTrainer.Profiles -> C:\Users\Prekzursil\Downloads\SWFOC editor\src\SwfocTrainer.Profiles\bin\Release\net8.0\SwfocTrainer.Profiles.dll
SwfocTrainer.Saves -> C:\Users\Prekzursil\Downloads\SWFOC editor\src\SwfocTrainer.Saves\bin\Release\net8.0\SwfocTrainer.Saves.dll
SwfocTrainer.Runtime -> C:\Users\Prekzursil\Downloads\SWFOC editor\src\SwfocTrainer.Runtime\bin\Release\net8.0-windows\SwfocTrainer.Runtime.dll
SwfocTrainer.App -> C:\Users\Prekzursil\Downloads\SWFOC editor\src\SwfocTrainer.App\bin\Release\net8.0-windows\SwfocTrainer.App.dll
SwfocTrainer.Tests -> C:\Users\Prekzursil\Downloads\SWFOC editor\tests\SwfocTrainer.Tests\bin\Release\net8.0-windows\SwfocTrainer.Tests.dll

Build succeeded.
0 Warning(s)
0 Error(s)

Time Elapsed 00:00:04.09
passed\n - Test run for C:\Users\Prekzursil\Downloads\SWFOC editor\tests\SwfocTrainer.Tests\bin\Release\net8.0-windows\SwfocTrainer.Tests.dll (.NETCoreApp,Version=v8.0)
Microsoft (R) Test Execution Command Line Tool Version 17.8.0 (x64)
Copyright (c) Microsoft Corporation. All rights reserved.

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.

Passed! - Failed: 0, Passed: 270, Skipped: 0, Total: 270, Duration: 2 s - SwfocTrainer.Tests.dll (net8.0)
passed ( tests)\n - SonarCloud + SonarCloud Code Analysis now pass on latest head.\n\n- Remaining hard blockers for full strict-zero closure:\n 1. Codecov 100/100 gate still failing at current baseline (, ) from .\n 2. Provider zero gates currently hard-fail in strict mode without configured credentials/secrets/vars (, , , , , ).\n 3. Codacy PR check is still being reconciled by provider side for latest head (last observed state in PR rollup remains ).\n\n- Branch protection has been updated to require the expanded strict-zero contexts on per plan.\n\nI will continue with the next wave once provider credentials/backlog policy inputs are available and Codacy finishes the newest analysis cycle.

Prekzursil and others added 3 commits March 3, 2026 05:57
Use resolved dotnet executable path instead of relying on plain command lookup so coverage collection behaves consistently in PR validation shells.

Co-authored-by: Codex <noreply@openai.com>
Force rebuild semantics for deep installed-chain matrices to avoid no-build artifact drift, and stage coverage collection under a temp root with explicit dotnet invocation handling.

Co-authored-by: Codex <noreply@openai.com>
Add fallback TRX resolution when expected files are delayed/renamed and keep deep-chain runs build-safe under explicit NoBuild requests.

Co-authored-by: Codex <noreply@openai.com>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

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

Labels

risk:high High-risk change; strict human gate required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant