feat(engine): add hasPlanSkippedSteps and document plan DAG helpers#3470
Conversation
Closes JSONbored#2298 Co-authored-by: Cursor <cursoragent@cursor.com>
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
|
Warning 🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨 ⏸️ Gittensory review result - manual review recommendedReview updated: 2026-07-05 08:41:00 UTC
⏸️ Suggested Action - Manual Review Review summary Nits — 4 non-blocking
Review context
Contributor next steps
Signal definitions
🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3470 +/- ##
=======================================
Coverage 93.03% 93.03%
=======================================
Files 297 298 +1
Lines 31202 31203 +1
Branches 11386 11386
=======================================
+ Hits 29028 29029 +1
Misses 1518 1518
Partials 656 656
🚀 New features to boost your workflow:
|
Closes #2298
Summary
hasPlanSkippedStepsin@jsonbored/gittensory-engine— returns whether any step in a plan DAG was skipped.packages/gittensory-engine/README.md(advances the engine README export-list deliverable in docs(miner-foundation): gittensory-miner and gittensory-engine README + package docs #2298).Test plan
COVERAGE_NO_THRESHOLDS=1 npx vitest run test/unit/plan-skipped.test.ts --coveragenpx diff-cover coverage/lcov.info --compare-branch=main --fail-under=99→ 100%npm run build --workspace @jsonbored/gittensory-engine && npm run build:minerMade with Cursor