Repository navigation
Clean up medium_structure_containment lens residue after #6831 - #6853
Merged
Merged
Conversation
gunbai-bot
Bot
changed the title
Contributor
|
Addressing review 39634 (non-blocking):
review 39636: No action items — findings none, verdict APPROVE. CI on — sent from swift-ibex-478 |
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
briansrls
added a commit
that referenced
this pull request
Jul 19, 2026
…ain-merge Addresses cursor review 39633 on the Phase-0/1 landing-record and the two batch-2 witness failures after merging main (which landed #6853/#6854): - shell_emission_model.dag (finding 1): reconcile the three "For/While refuse by design / NOT in scope / While stays emitter-unsupported" rows — the 2026-07-03 census's "zero pre-runtime While sites" was superseded by #6832, which landed While/BoundedPoll emit for the ci_floor_peak_emit readiness-poll. - roadmap_authority.dag (finding 2): 6-shell-intent-phase1 set done: false / operator-sign-pending (merged #6832, not operator-signed — no forged sign, mirrors 6-shell-slice2). Reverted the redundant #6832 add to 2-debash-orchestration's prs (restored the render witness_emit_refs_from_binding pins as "(#6106)"). - Regenerated DESIGN.md / ROADMAP.md / shell-emission-model.md (drift gate green). live_deploy_emit_holds was a stale-base failure, resolved by the main-merge. Verified green-by-execution: roadmap_authority_witnesses, live_deploy_emit_holds, generated_artifact_drift_witnesses all true. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
6 tasks
briansrls
added a commit
that referenced
this pull request
Jul 19, 2026
…des #6852) (#6878) * Record shell→intent Phase 1 landing in authority carriers (#6832). Implementation (While/BoundedPoll/general Retry emit) merged via #6832; update design doc, roadmap, and open-thread status so workers are not re-dispatched onto finished Phase 1. Co-authored-by: Cursor <cursoragent@cursor.com> * Fix #6852 CI: reconcile stale While-rows + roadmap done-state after main-merge Addresses cursor review 39633 on the Phase-0/1 landing-record and the two batch-2 witness failures after merging main (which landed #6853/#6854): - shell_emission_model.dag (finding 1): reconcile the three "For/While refuse by design / NOT in scope / While stays emitter-unsupported" rows — the 2026-07-03 census's "zero pre-runtime While sites" was superseded by #6832, which landed While/BoundedPoll emit for the ci_floor_peak_emit readiness-poll. - roadmap_authority.dag (finding 2): 6-shell-intent-phase1 set done: false / operator-sign-pending (merged #6832, not operator-signed — no forged sign, mirrors 6-shell-slice2). Reverted the redundant #6832 add to 2-debash-orchestration's prs (restored the render witness_emit_refs_from_binding pins as "(#6106)"). - Regenerated DESIGN.md / ROADMAP.md / shell-emission-model.md (drift gate green). live_deploy_emit_holds was a stale-base failure, resolved by the main-merge. Verified green-by-execution: roadmap_authority_witnesses, live_deploy_emit_holds, generated_artifact_drift_witnesses all true. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * WIP: shell -> dag --------- Co-authored-by: Brian Searls <briansearls1@gmail.com> Co-authored-by: Cursor <cursoragent@cursor.com> Co-authored-by: Brian Searls <briansrls@gunb.ai> Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
#6831 already removed the executable
medium_structure_containment/bash_composition_recognizerlens (module, registry row, witnesses, v1 builtins, and CI enrollment). Nothing in this PR re-deletes that work — it scrubs surviving prose residue that still named the lens as live enforcement:medium_structure_census.rs→parsed_dag_file.rs(parse-only helper; no census/lens logic)medium_structure_exception_rosterreferences from dissolution-trigger strings and the shell_bash_runner witnessNo new enforcement mechanism replaces the deleted lens.
realization_vocabulary_containmentremains the live shell-vocab gate.Test plan
cargo check -p v1-compiler(module rename)origin/main:src/v2/lens/medium_structure_containment.dag, registryMediumStructureContainmentid,commit_workflowenrollment, v1medium_structure_*_livebuiltins