Skip to content

test(incremental): prove node deletion failures surface#68

Open
100yenadmin wants to merge 1 commit into
reconcile/r07-large-incremental-harnessfrom
reconcile/r08-nodebatch-disposition
Open

test(incremental): prove node deletion failures surface#68
100yenadmin wants to merge 1 commit into
reconcile/r07-large-incremental-harnessfrom
reconcile/r08-nodebatch-disposition

Conversation

@100yenadmin

Copy link
Copy Markdown
Member

Summary

  • add a real LadybugDB regression contract for unexpected node-table deletion failures
  • preserve the documented zero-match and missing-embedding-table behavior
  • pair the contract with the retained R07 subprocess and escalation evidence

Evidence

  • lbug-delete-nodes-for-files: 4/4 passed
  • incremental-escalation-gate: 6/6 passed
  • incremental-orchestration: 12/12 passed
  • Prettier check passed
  • GitNexus detect_changes was attempted with the explicit worktree and safely refused because the registered MCP repository is a different canonical clone; no index refresh was performed

Scope

This PR changes one integration test file. It does not alter production deletion behavior, force rebuilds, mutate a live index, or modify the preserved worktree.

Tracks #47. Dispositions #37 after current-head CI is green.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Jul 13, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 564abcc2-6305-4dc7-9b2c-b9fe54efb93d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch reconcile/r08-nodebatch-disposition

Comment @coderabbitai help to get the list of available commands.

@evaos-code-review-bot

evaos-code-review-bot Bot commented Jul 13, 2026

Copy link
Copy Markdown

evaOS review status: stale head

PR: #68 - test(incremental): prove node deletion failures surface
Head: 670f4d8c9abfcdf4ce76519f890d1f5e1eab7bf0
Updated: 2026-07-13T12:32:58.217Z

evaOS review stopped because this queued head is no longer the live PR head.

Automation note: agents should wait for this comment to reach completed, stale_head, closed_or_merged_before_review, skipped, or failed before treating evaOS review as settled for this head. provider_deferred means evaOS still intends to retry.

PR URL: #68

Details: Superseded by a newer PR head.

@100yenadmin 100yenadmin force-pushed the reconcile/r08-nodebatch-disposition branch from 670f4d8 to f767e35 Compare July 13, 2026 12:25
@100yenadmin 100yenadmin force-pushed the reconcile/r07-large-incremental-harness branch from f5357e1 to 22055c6 Compare July 13, 2026 12:25
@evaos-code-review-bot

evaos-code-review-bot Bot commented Jul 13, 2026

Copy link
Copy Markdown

evaOS review status: failed

PR: #68 - test(incremental): prove node deletion failures surface
Head: f767e35e0683dfec5481719d0455f8cb3a312adb
Updated: 2026-07-13T12:59:05.531Z

evaOS review failed for this head and needs retry or operator attention.

Automation note: agents should wait for this comment to reach completed, stale_head, closed_or_merged_before_review, skipped, or failed before treating evaOS review as settled for this head. provider_deferred means evaOS still intends to retry.

PR URL: #68

Details: Review failed; see bot evidence for operator-only details.

@github-actions

github-actions Bot commented Jul 13, 2026

Copy link
Copy Markdown

CI Report

All checks passed

Pipeline Status

Stage Status Details
✅ Typecheck success tsc --noEmit
✅ Tests success unit tests, 3 platforms
✅ E2E success gitnexus-web changes only

Test Results

Tests Passed Failed Skipped Duration
14537 14475 0 62 21s

✅ All 14475 tests passed

62 test(s) skipped — expand for details

Code Coverage

Tests

Metric Coverage Covered Base Delta Status
Statements 80.7% 53912/66802 N/A% 🟢 ████████████████░░░░
Branches 67.7% 32919/48618 N/A% 🟢 █████████████░░░░░░░
Functions 86.75% 6214/7163 N/A% 🟢 █████████████████░░░
Lines 84.23% 48091/57092 N/A% 🟢 ████████████████░░░░

📋 View full run · Generated by CI

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