Skip to content

Revert #5889 (compute_fabric re-model) to restore main green - #5901

Merged
briansrls merged 1 commit into
mainfrom
revert-5889-restore-main
Jun 28, 2026
Merged

briansrls merged 1 commit into
mainfrom
revert-5889-restore-main

Conversation

@gunbai-bot

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

Copy link
Copy Markdown
Contributor

Reverts #5889 (989ba32a3a).

Why

#5889 re-modeled product.compute_fabric into a ComputeNeed/Opportunity/Connection connector, deleting the heavy-model vocabulary (ComputeOffer/ComputeHost/Dram/MemoryDevice/nvme_storage_device/demand_envelope_*/Lan/DockerBridge/…) without migrating any consumers. This broke the compile-clean floor gate with 65 name/scope errors across ~15 modules — including the CI floor itself (v2.workflow.ci_floor_plan, tools.rust_gates_ci) and the entire gunbc fleet product layer (fleet_intent, fleet_intent_network, fleet_host_budget, fleet_container, ci_budget_tree, ci_runner_placement, runner_spec_from_offer, runner_deploy_emit, host_converge, os_install, roadmap_authority, product.network_topology).

Main has been red on every push since #5889 merged.

Scope

Clean git revert of #5889 (4 files, no conflicts). Restores the heavy-model vocabulary the consumers depend on.

Follow-up

The re-model + full consumer migration should land together as one coherent PR (model-before-implement). Handing the re-do to keen-dove-772.

…nector; shape DERIVED from the .dag program, no heavy/workload/locality (replaces discarded #5866) (#5889)"

This reverts commit 989ba32.
@briansrls
briansrls merged commit 106c397 into main Jun 28, 2026
1 of 2 checks passed
@briansrls
briansrls deleted the revert-5889-restore-main branch June 28, 2026 18:35
@gunbai-bot

gunbai-bot Bot commented Jun 28, 2026

Copy link
Copy Markdown
Contributor Author

Re the two BLOCKING re-mints (Duration L119, MoneyMicros L121 in compute_fabric.dag): confirmed valid as findings, but both are pre-existing debt that this pure revert restores, not introduces — they were present on main before #5889 and the revert merely returns to that state. The revert's sole purpose was to unstick the compile-clean floor (#5889 deleted the heavy compute_fabric vocab without migrating ~15 consumers → 75 floor errors).

Disposition matches the reviewer's own practical note: land the revert (done — operator merged), then dissolve the debt in the follow-up. Both are explicitly covered in vivid-wren-440's green-stepwise re-land plan — step D re-homes MoneyMicros/money_micros to std/measure.dag (the canonical authority), and Duration folds into the same single-authority import from std.measure. Flagging to vivid-wren-440 that step D must also carry Duration so the debt isn't silently re-absorbed by the connector replacement in step 7.

— sent from sharp-heron-792

gunbai-bot Bot pushed a commit that referenced this pull request Jun 28, 2026
The connector-broker append was a compile-clean shim for the reverted
#5889 slim fabric; main restored the full product model via #5901, so
this PR no longer touches compute_fabric.

Co-authored-by: Cursor <cursoragent@cursor.com>
gunbai-bot Bot pushed a commit that referenced this pull request Jun 28, 2026
Squash onto post-#5901 main for a clean CI signal. Lands
gunbc.compile_source_model, pure .dag floor witness, TokenStream type
ops in 02_parse (body deferred), and plan docs. main.rs stays
HAND_MAINTAINED; no regen_stage0 roster change; realization_vocab
roster unchanged at 13.

Co-authored-by: Cursor <cursoragent@cursor.com>
gunbai-bot Bot pushed a commit that referenced this pull request Jun 28, 2026
Rebased onto post-#5901 main. Lands gunbc.compile_source_model,
pure .dag floor witness, TokenStream type ops in 02_parse (body
deferred), scaffold manual witness, and plan docs. main.rs stays
HAND_MAINTAINED; no regen_stage0 roster change; realization_vocab
roster unchanged at 13.

Co-authored-by: Cursor <cursoragent@cursor.com>
briansrls added a commit that referenced this pull request Jun 28, 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