Skip to content

Regime-2 emission shared fold - collapse boutique yaml gitignore runner_deploy serializers into one render over a shared std layout IR - model-checkpoint-gated to warm-lark-306 - #5614

Closed
gunbai-bot[bot] wants to merge 9 commits into
mainfrom
session/tidy-crane-536

Conversation

@gunbai-bot

@gunbai-bot gunbai-bot Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Auto-opened by session-dashboard for session tidy-crane-536.
Pushing to session/tidy-crane-536 advances this PR.

Worker attestation

Before flipping this PR to ready for review, confirm each item:

  • Title describes the change (not the session id or branch).
  • PR body summarises what and why (replace the TODO below).
  • Tests run: name the command (e.g. npm test, cargo test) and the result.
  • If this closes a work item, the body contains a Closes #N directive.
  • No commits on this branch are surprises (no fork/cherry-pick I did not make).
  • No secrets / credentials / large binaries staged.

Summary

TODO: replace this paragraph with one or two sentences naming the change and its motivation. Reviewers read this first.

Test plan

  • TODO: list the commands that ran (or "no tests changed; relied on CI") and the outcome.

briansrls and others added 9 commits June 22, 2026 22:58
…latent §5 fail-open)

The regime-2 refactor's emit_inline_scalar_text dropped emit_scalar_string's
multiline '|' branch, so a multiline YamlString outside mapping-value position
(top-level scalar, bare sequence element) emitted embedded raw newlines —
broken YAML, a §5 fail-open. Latent (serialize_yaml's only consumer is ci.yml,
which never hits those positions) but a real faithfulness regression vs the old
serialize_yaml.

Fix (behavior parity): route multiline strings through a shared multiline_block_doc('|')
helper in the mapping-value, bare-sequence-element, and top-level-scalar arms;
exclude multiline strings from all_scalars_flowable so they take the block path,
not the (also-broken) flow path. ci.yml + .gitignore stay byte-identical
(mapping-value path unperturbed); new yaml_multiline_block_witness asserts the
block literal in all three positions and goes RED on the embedded-newline form.

Bundled: repoint stale prose in docs/plans/emission-ingestion-inverse.md (named
the deleted emit_yaml_value/emit_mapping_body).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@briansrls
briansrls marked this pull request as ready for review June 23, 2026 05:02
@gunbai-bot

gunbai-bot Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

Closing as superseded. This PR's stated purpose — collapse the boutique yaml/gitignore/runner_deploy serializers into one render over a shared std.layout IR — landed on main independently as #5581 (40220d46b3: "Regime-2 emission: collapse yaml/gitignore/runner_deploy serializers into one std.layout render fold"), with the follow-on yaml wildcard→total-match dissolution in #5612 (953fe0f8d1).

Integrating current main into this branch produces a byte-identical tree (git diff origin/main HEAD is empty across all touched files, including dsl/std/layout.dag, dsl/extdeps/formats/yaml.dag, dsl/gunbc/{gitignore,runner_deploy}_emit.dag, dsl/shared/dag_util.dag, dsl/std/types.dag, src/v1/stage0/src/std_types.rs, src/v2/lens/medium_structure_containment.dag, and the three witness tests). There is no net change left to merge, so the PR cannot be made meaningfully mergeable — the work is already in main.

@gunbai-bot gunbai-bot Bot closed this Jun 24, 2026
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