Skip to content

feat(smart-rma): add browser parser - #12

Merged
margrop merged 1 commit into
mainfrom
agent/p3-002-smart-parser
Jul 25, 2026
Merged

margrop merged 1 commit into
mainfrom
agent/p3-002-smart-parser

Conversation

@margrop

@margrop margrop commented Jul 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • define the versioned smart-rma-parse-result-v1 contract and stable failure model
  • add a deterministic browser parser for ATA, NVMe, unsupported bridges, key counters, signals, and missing fields
  • launch the indexable /smart-rma/ Alpha workbench with all seven fully synthetic P3-001 samples
  • add responsive/accessibility styles, static production-output checks, tests, and operator documentation
  • mark P3-002 complete and recommend P3-003 next

Privacy and cost

  • raw SMART text remains in the current page memory only
  • no URL, Local Storage, Analytics, logging, network, or AI side effects
  • parse results exclude raw text, model text, serial numbers, and WWN
  • this task makes no AI calls and adds no runtime AI cost

Validation

  • npm run validate
  • Astro/TypeScript: 0 errors, warnings, or hints
  • Vitest: 19 files and 216 tests passed
  • static build: 5 pages, including /smart-rma/
  • all static output contracts passed

Deliberate limits

  • P3-003 still owns complete serial number, WWN, host name, and IP redaction for real pasted input
  • P3-004 still owns health/warning/critical/unknown classification
  • this parser does not provide AI explanations, exports, or a vendor warranty/RMA decision

Next

Implement P3-003 as a separate PR with fail-closed redaction and sink-boundary tests.

@margrop
margrop marked this pull request as ready for review July 25, 2026 13:24
@margrop
margrop merged commit dd77f96 into main Jul 25, 2026
1 check passed
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