feat: implement reporting storage repository with adapters and bulk backfill

[Copilot]

Feature Pull Request

This repository enforces changelog, release, and label automation for all PRs and issues.
See the organisation-wide Automation Governance & Release Strategy for contributor rules.

Linked issues

Changelog

Added

This PR introduces the Reporting Agent v2 Phase 3.2 storage layer for persisted metadata, including a repository abstraction, SQL/NoSQL adapter model, transactional semantics, and bulk historical backfill support.

• New storage module: scripts/metrics/integrations/reporting-storage-adapter.js
• New fixtures: scripts/metrics/integrations/__tests__/fixtures/reporting-storage-fixtures.json
• New test suite: scripts/metrics/integrations/__tests__/reporting-storage-adapter.test.js (30+ coverage target addressed)

Changed

• Repository abstraction
  • ReportingStorageRepository now encapsulates storage operations behind an adapter contract.
• Adapter pattern (SQL/NoSQL)
  • Added SqlMetadataAdapter and NoSqlMetadataAdapter with a shared in-memory transactional base.
• CRUD and query operations
  • Added create/read/update/delete plus criteria-based query with sort, offset, and limit handling.
• Bulk backfill
  • Added batched backfill with upsert/non-upsert modes, atomic/non-atomic behaviour, and structured partial error reporting.
• Transaction support
  • Added begin/commit/rollback with rollback-failure propagation and contextual error wrapping.

Example usage:

const repository = new ReportingStorageRepository({ adapter: "sql" });

repository.bulkBackfill(records, {
  batchSize: 100,
  transaction: true,
  atomic: true,
  upsert: true,
});

Fixed

• Reduced duplicated adapter filtering logic by centralising criteria matching in the shared base adapter.
• Clarified and hardened error behaviour for rollback/commit failure paths in bulk operations.

Removed

• No removals.

---

Checklist (Global DoD / PR)

• All AC met and demonstrated
• Tests added/updated (unit/E2E as appropriate)
• Accessibility checklist completed (where relevant):
  • Semantic HTML and heading order verified
  • Keyboard navigation and visible focus states verified
  • ARIA used only where needed
  • Contrast and non-colour cues reviewed (WCAG 2.2 AA or higher)
• Docs/readme/changelog updated (if user-facing)
• Security checklist completed (where relevant):
  • Untrusted input validated and sanitised
  • Output escaped for its rendering context
  • Privileged actions enforce nonce and capability checks
  • No secrets/sensitive data introduced; OWASP risks reviewed
• Code/design reviews approved
• CI green; linked issues closed; release notes prepared (if shipping)

---

• Fixes Reporting Agent v2 Phase 3.2: Storage Layer Implementation #2036

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[github-actions]

🔍 Reviewer Summary for PR #2488

CI Status: ❌ pending
Files changed: 3

Recommendations

• CI checks not green

[github-actions]

🔍 Reviewer Summary for PR #2488

CI Status: ❌ pending
Files changed: 3

Recommendations

• CI checks not green

[github-actions]

🚫 This PR description is missing required template content.

Missing required section(s): Linked issues, Global DoD checklist

Please update the PR body using one of the repository PR templates:

• https://github.com/lightspeedwp/.github/blob/develop/.github/pull_request_template.md
• https://github.com/lightspeedwp/.github/tree/develop/.github/PULL_REQUEST_TEMPLATE

Empty placeholders, unchecked checklist boxes, and stub issue references do not count.

[github-actions]

🔍 Reviewer Summary for PR #2488

CI Status: ❌ pending
Files changed: 3

Recommendations

• CI checks not green

[github-actions]

⚠️ Branch Naming Issue

This PR's branch name copilot/reporting-agent-v2-storage-layer-implementation doesn't follow the standard pattern.

Standard pattern: {type}/{scope}-{title} (e.g., feat/something, fix/bug-name)

Based on linked issue type: feature

Correct template should be: pr_feature.md

No action required — this is informational. Future PRs should use the correct branch name.

See Branching Strategy for more info.

[github-actions]

🔍 Reviewer Summary for PR #2488

CI Status: ❌ pending
Files changed: 3

Recommendations

• CI checks not green