Skip to content

refactor(core): cpp23 read_json_model.c → .cpp (ADR-0846) - #264

Closed
lusoris wants to merge 1 commit into
masterfrom
refactor/cpp23-read-json-model-clean-20260530
Closed

lusoris wants to merge 1 commit into
masterfrom
refactor/cpp23-read-json-model-clean-20260530

Conversation

@lusoris

@lusoris lusoris commented May 30, 2026

Copy link
Copy Markdown
Contributor

Recreates closed PR #215's 7-file change. C++23 conversion of read_json_model with isolated read_json_model_cpp23_lib static lib pattern.

no docs needed: internal C++23 conversion; ADR-0846 + AGENTS.md updated in same PR.

Test plan

  • 7 files only
  • CI: build green; fast suite 49/49

Deep-dive deliverables (ADR-0108)

  • Research digest — no digest needed: Wave 8 mechanical port
  • Decision matrix — ADR-0846 (in diff)
  • AGENTS.md invariant note — core/AGENTS.md (in diff)
  • Reproducer — meson test -C build --suite=fast
  • Changelog fragment — changelog.d/changed/0846-cpp23-read-json-model.md (in diff)
  • Rebase-notes entry — docs/rebase-notes.md (in diff)

Lint clean (CLAUDE §12 r12)

  • 7 files touched

State drift (CLAUDE §12 r13)

  • No bug-tracking change

FFmpeg-patch sync (CLAUDE §12 r14)

  • No public-API change — internal C++23 conversion

🤖 Generated with Claude Code

Recreates closed PR #215's 7-file change. cpp23 Wave 8: read_json_model
ported to C++23 with isolated read_json_model_cpp23_lib static lib.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@lusoris
lusoris enabled auto-merge (squash) May 30, 2026 07:00
@lusoris

lusoris commented May 30, 2026

Copy link
Copy Markdown
Contributor Author

Real bug — cpp23 read_json_model conversion broke nearly every Build job + Sanitizers + Netflix Golden. Needs proper investigation, not safe for unattended train. Skipping.

@lusoris lusoris closed this May 30, 2026
auto-merge was automatically disabled May 30, 2026 07:21

Pull request was closed

@lusoris
lusoris deleted the refactor/cpp23-read-json-model-clean-20260530 branch June 4, 2026 10:26
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