Skip to content

feat(B-0068.1): smallest safe slice — Forge CLI + Ollama research child row (re-decomposed from umbrella)#2430

Merged
AceHack merged 5 commits into
mainfrom
claim/B-0068-decompose-smallest-slice-riven-2026-05-10
May 10, 2026
Merged

feat(B-0068.1): smallest safe slice — Forge CLI + Ollama research child row (re-decomposed from umbrella)#2430
AceHack merged 5 commits into
mainfrom
claim/B-0068-decompose-smallest-slice-riven-2026-05-10

Conversation

@AceHack
Copy link
Copy Markdown
Member

@AceHack AceHack commented May 10, 2026

Summary

Decomposed B-0068 (P2 local-AI umbrella) into smallest dependency-ordered atomic child backlog rows.
One bounded step per rules: created first child B-0068.1 (research-only S slice on Forge CLI + Ollama bridge).

Re-decomposition note: B-0068 umbrella assumed broad → first child is pure research (WebSearch + doc) to keep velocity. Future children: hardware audit (TS), smoke-test, direct-integration matrix. Always re-decompose during build.

Child row created

docs/backlog/P2/B-0068.1-forge-cli-ollama-research-slice-aaron-2026-05-10.md

Focused checks (per task rule)

  • Root dotnet build -c Release: transient MSB4166 (MSBuild parallel flake, 0 code warnings/errors)
  • Worktree focused dotnet build src/Core -c Release: expected assets missing (fresh worktree, no code impact)
  • rg confirmed no prior Forge/Ollama in P2 backlog
  • Worktree + pushed claim branch used; root checkout untouched

Rules followed

  • Dedicated worktree + pushed claim branch before write
  • TS over bash (no .sh)
  • Prefer F#/TS over docs (research slice defers code; next child will be TS hardware audit)
  • Exactly one bounded step + open PR
  • Substrate-or-it-didn't-happen: child row is durable backlog substrate

Co-Authored-By: Grok noreply@x.ai

Made with Cursor

AceHack and others added 3 commits May 10, 2026 02:15
… catalog PRs

TS2322 (candidate-schema.ts:52): f2Strength 'strong-but-looser' not assignable
to union 'weak'|'moderate'|'strong' — change to 'moderate'; qualifier moved
to notes field.

TS2375 (mythology-catalog-schema.ts:451): explicit `sourceMemory: undefined`
violates exactOptionalPropertyTypes=true — remove line; omission is correct
for absent optional properties.

Both errors pre-existed in main via #2423/#2424; first surfaced by tsc lint
running against PR #2426's merge context.

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

The notes field contradicted textualAnchors listing two Eddas by saying 'second anchor pending'. Reworded to specify additional independent anchor beyond the Eddas is pending for F2.

Co-Authored-By: Grok <noreply@x.ai>
Co-authored-by: Cursor <cursoragent@cursor.com>
…ld row (re-decomposed from umbrella)

Decomposed B-0068 umbrella per "if too broad, decompose before implementation".
One bounded step: first atomic research child (S-effort, no code).
Dependency-ordered start: research → hardware audit → install/smoke → direct paths.

Focused checks included in child doc body.
Co-Authored-By: Grok <noreply@x.ai>

Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI review requested due to automatic review settings May 10, 2026 06:56
@AceHack AceHack enabled auto-merge (squash) May 10, 2026 06:57
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Decomposes the P2 local-AI umbrella (B-0068) by adding an initial research-only child backlog row (B-0068.1), while also making small consistency edits in the mythology-resonance TypeScript schemas.

Changes:

  • Added new backlog child row B-0068.1 describing a bounded Forge CLI + Ollama research slice.
  • Adjusted mythology-resonance candidate seed metadata (F2 strength value + notes) to match the schema.
  • Removed an explicit sourceMemory: undefined entry from the mythology resonance seed catalog (optional field cleanup).

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
tools/resonance/mythology-catalog-schema.ts Removes an explicit undefined assignment for an optional field in the seed catalog.
tools/mythology-resonance/candidate-schema.ts Updates Heimdallr seed f2Strength value and elaborates notes.
docs/backlog/P2/B-0068.1-forge-cli-ollama-research-slice-aaron-2026-05-10.md Adds a new P2 backlog child row defining the research-only scope and intended outputs.

Comment thread tools/mythology-resonance/candidate-schema.ts
@AceHack AceHack merged commit feb221c into main May 10, 2026
26 checks passed
@AceHack AceHack deleted the claim/B-0068-decompose-smallest-slice-riven-2026-05-10 branch May 10, 2026 07:12
AceHack added a commit that referenced this pull request May 13, 2026
…ed row → B-0068.4 (#3057)

* fix(backlog): resolve B-0068.1 ID collision — renumber Aaron-attributed row to B-0068.4

First per-collision cleanup from the B-0451 sweep (12 duplicate-ID
groups surfaced by `tools/bg/audit-duplicate-row-ids.ts` in PR #3056).

The two B-0068.1 rows:

- `B-0068.1-forge-cli-ollama-research-slice-aaron-2026-05-10.md`
  — filed first (2026-05-10, PR #2430); scope: "Forge CLI + Ollama
  harness integration research slice"
- `B-0068.1-forge-cli-ollama-research-xs-riven-2026-05-11.md`
  — filed second (2026-05-11, PR #2650); scope: "Forge CLI + Ollama
  bridge research pass (WebSearch + capability matrix, XS)"

Per the per-collision resolution rule (keep the row with external
references):

- The B-0068 parent row's body description ("B-0068.1 (XS, P2, root):
  Forge CLI + Ollama bridge research (WebSearch + matrix). Unblocks
  B-0068.3.") describes RIVEN'S scope.
- Sibling rows `B-0068.2` and `B-0068.3` reference "B-0068.1" in their
  depends_on / composes_with — semantically referring to Riven's
  decomposition (parallel atomic XS series).
- Aaron's row has NO external references (verified via grep).

→ Keep Riven's row at B-0068.1; renumber Aaron's to next-free
B-0068.4 with `renumbered_from: B-0068.1` + reason recorded.

This bends the "first-merged-wins" rule in favor of the
external-references rule, matching PR #3053's B-0444 resolution
precedent (keep the more-referenced row, even when later-filed).

Empirical effect (against pre-PR-#3056 origin/main):

  Duplicate-ID groups: 12 → 11

Once PR #3056 (the audit tool) merges, future Otto can verify via
`bun tools/bg/audit-duplicate-row-ids.ts`.

Co-Authored-By: Claude <noreply@anthropic.com>

* shard(tick): 2329Z — first B-0451 cleanup; B-0068.1 collision resolved

Records: PR #3049 (Lior 429 fix) merged. PR #3056 (audit tool +
B-0451) still wait-ci. Speculative pickup: started B-0451 per-collision
cleanup with the simplest collision (B-0068.1 within-priority pair).
Renumbered Aaron's row to B-0068.4; Riven's keeps original ID due to
external references. Procedure demonstrated for remaining 11 groups.

Co-Authored-By: Claude <noreply@anthropic.com>

---------

Co-authored-by: Claude <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.

2 participants