fix(nip56): address post-merge review findings on report ingestion

[Priyanshubhartistm]

Description

Following up on #770 after a collaborator review on the merged PR turned up four real bugs in report ingestion: dead rows for targetless reports, silent truncation of multi-target reports, non-atomic inserts across a report's rows, and the first report after a restart blocking on a cold WoT graph rebuild. Fixed all four.

Related Issue

Closes #785

Motivation and Context

All four bugs were already live on main

[changeset-bot]

🦋 Changeset detected

Latest commit: 2277916

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
nostream	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[greptile-apps]

Confidence Score: 5/5

The PR appears safe to merge; the unnecessary Redis connection at boot is non-blocking.

Findings

1.  Redis connects with WoT disabled ▶

Summary

The PR makes report-row insertion transactional, skips targetless rows, raises the target cap, and starts WoT graph warm-up at worker boot.

• The boot call also opens Redis when WoT is disabled.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Worker boot] --> B[getCache]
  B --> C[Redis adapter connects]
  C --> D[WoT factory]
  D --> E{WoT enabled?}
  E -- Yes --> F[Start graph warm-up]
  E -- No --> G[Skip warm-up]

Loading

_{Reviews (2) · Last reviewed commit: "fix(nip56): WoT warm-up and target cap"}

[coveralls]

Coverage is 72.466% — Priyanshubhartistm:fix/nip56-report-ingestion-gaps into cameri:main. No base build found for cameri:main.