Skip to content

dev learnings family: consolidate the read path (knowledge family retires) and tighten distribution - #80

Merged
JPDuchesne merged 2 commits into
mainfrom
ai/74-dev-learnings-family-consolidate-the-rea
Aug 1, 2026
Merged

dev learnings family: consolidate the read path (knowledge family retires) and tighten distribution#80
JPDuchesne merged 2 commits into
mainfrom
ai/74-dev-learnings-family-consolidate-the-rea

Conversation

@d3mlabs-ai-flow

Copy link
Copy Markdown
Contributor

Implements #74.

Requested by @JPDuchesne.

Closes #74

…ledge family retires) and tighten distribution

Co-authored-by: JPDuchesne <2636122+JPDuchesne@users.noreply.github.com>
@codecov

codecov Bot commented Aug 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@JPDuchesne

JPDuchesne commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

/build let's fix code coverage

/build — committed fd2b437.

Fixed the Codecov patch-coverage gap by exercising the previously untested status-report branches of the learnings accessor — no production code changes were needed, only tests. Getting the suite to run at all required diagnosing that the runner shell leaks the nested .ai-flow checkout's bundler environment (BUNDLE_GEMFILE, RUBYOPT, GEM_*), which fails the repo's Ruby pin, plus refreshing the per-machine install stamp via dev install-deps; that diagnosis was captured as a new repo-local learning toolchain/ai-flow-bundler-env-leak (indexed in the learnings index, detail skill under .cursor/skills/learnings/ai-flow-bundler-env-leak/) — it concerns the ai-flow runner environment rather than this repo's code, so it may be an org-promotion candidate later. Note: dev tc fails locally only because Sorbet sees duplicate gem RBIs from the untracked .ai-flow/ checkout; CI is unaffected.

🧠 drafted 1 learning in a learning proposal PR: #82

  • ai-flow-bundler-env-leak

⚙️ workflow run · model: claude-fable-5-high

Co-authored-by: JPDuchesne <2636122+JPDuchesne@users.noreply.github.com>
@JPDuchesne
JPDuchesne merged commit 48d1ae5 into main Aug 1, 2026
5 checks passed
@JPDuchesne
JPDuchesne deleted the ai/74-dev-learnings-family-consolidate-the-rea branch August 1, 2026 20:56
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.

dev learnings family: consolidate the read path (knowledge family retires) and tighten distribution

1 participant