Skip to content

CI floor profiling run: fractal Gantt chart on top of #5757 baseline - #5785

Merged
briansrls merged 4 commits into
mainfrom
session/valiant-deer-861
Jun 25, 2026
Merged

briansrls merged 4 commits into
mainfrom
session/valiant-deer-861

Conversation

@briansrls

Copy link
Copy Markdown
Contributor

Auto-opened by session-dashboard for session valiant-deer-861.
Pushing to session/valiant-deer-861 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.

Brian Searls and others added 4 commits June 24, 2026 23:57
…g receipt plan doc

- `ClaimResult` gains `wall_nanos`, `resolve_nanos`, `corpus_resolve_nanos`,
  `corpus_eval_nanos`, `corpus_witnesses` fields (WIP had the struct; this
  commit wires the collection side)
- `run_shared_entry_claims` now times the resolve phase (Instant before
  resolve_entry_graph) and each claim eval (Instant around run_claim);
  resolve cost is attributed to the first claim in the SharedClaims group
  (the group resolves once, cost charged once)
- `run_discovery_batch_node` threads `DiscoverySummary.total_resolve_nanos`
  and `total_measured_nanos` into the returned ClaimResult so the corpus
  serial-sum breakdown appears in the Gantt tree
- `run_walk` gains batch-level Instant timing; collects BatchRecord per batch
- `emit_gantt` emits a three-level `[gantt]` tree to stderr when
  `GUNBC_FLOOR_GANTT=1`: total → per-batch → per-claim (resolve + eval
  separate; discovery node shows serial-sum resolve/eval vs batch wall)
- `dsl/gunbc/plans/ci_floor_fractal_gantt.dag`: measurement receipt plan doc
  with the fractal Gantt from CI run 28136318404 (2026-06-24, post-#5757),
  top-3 hot spot analysis, and quick-win vs architectural classification

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…stration drift fix

Run main_wet on generated_artifact_gate.dag to produce docs/plans/ci-floor-fractal-gantt.md
(the new Gantt plan's generated projection) and refresh docs/plans/host-effect-orchestration.md
whose Phase A content drifted. Fixes generated_artifact_drift_gate_passes which was red on main.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…hority

Add authored_doc entry for the fractal Gantt receipt in roadmap_authority.dag
(Floor wall-clock section) so docs/plans/ci-floor-fractal-gantt.md is reachable
from ROADMAP.md. Regen ROADMAP.md. Fixes doc_graph_has_no_orphan_docs.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@briansrls
briansrls marked this pull request as ready for review June 25, 2026 01:05
@briansrls
briansrls merged commit 1ae566d into main Jun 25, 2026
4 of 6 checks passed
@briansrls
briansrls deleted the session/valiant-deer-861 branch June 25, 2026 02:15
briansrls added a commit that referenced this pull request Jun 25, 2026
…mplete #5790) (#5796)

#5790 restored host_effect_orchestration.dag Phase A/B to [LANDED #5756] but did
NOT regenerate the committed .md projection. Meanwhile #5785 (a CI floor profiling
run) had regenerated the .md from the THEN-stale .dag, leaving the committed .md at
[now, parallel]. Result: .dag=LANDED vs committed .md=now-parallel — the drift
gate stayed RED on main, blocking every open PR (the whole de-fork wave).

This is a pure regen-on-main: the .dag authority is already correct and settled
(#5790), so regenerating its projection makes projection(.dag) == committed .md
with the LANDED text preserved. No content decision — just running main_wet on the
already-correct authority. Verified: regen changes ONLY this file; .md now carries
[LANDED #5756]; gate green.

Co-authored-by: Brian Searls <briansearls1@gmail.com>
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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