Skip to content

Harden PetVault skill architecture and report artifact pipelineFix/structure and report sanitizer - #1

Merged
Kingslayer-bot merged 2 commits into
mainfrom
fix/structure-and-report-sanitizer
Jul 8, 2026
Merged

Harden PetVault skill architecture and report artifact pipelineFix/structure and report sanitizer#1
Kingslayer-bot merged 2 commits into
mainfrom
fix/structure-and-report-sanitizer

Conversation

@Kingslayer-bot

Copy link
Copy Markdown
Owner

Summary

This PR hardens the PetVault skill architecture and report artifact pipeline after final release validation.

Key changes

  • Added .agents/, AGENTS.md, eval cases, examples, and release validation docs.

  • Unified forbidden-term registry and report sanitizer to prevent internal leakage.

  • Extracted deterministic modules:

    • billing_ops.py
    • material_ops.py
    • latex_ops.py
    • manifest_ops.py
    • pdf_ops.py
    • bill_render_ops.py
    • agent_registry_loader.py
  • Added reconstructed bill sections instead of raw invoice image embedding.

  • Added golden text snapshots and report artifact tests.

  • Added six synthetic user-flow examples:

    • bill_explain
    • knowledge_query
    • emergency_guardrail
    • insurance_boundary
    • travel_care
    • product_fit
  • Updated:

    • SKILL.md
    • AGENTS.md
    • docs
    • CI workflow
    • CHANGELOG

Final Validation

Final release verdict: GO.

Validation rounds:

  • Round 1: 10 validation cases prepared; baseline tests passed.
  • Round 2: Agent A user-side validation score 4.75/5; Agent B developer validation passed.
  • Round 3: A/B integrated validation agreement rate 100%.
  • Round 4: final regression, git hygiene, and GitHub push passed.

Test Result

Command:

python -m unittest discover -s pet-vault-skill/tests -p "test_*.py" -v

codex added 2 commits July 8, 2026 11:25
- Add release validation corpus (10 synthetic public bill scenarios)
- Run A/B parallel validation (user-side + developer-side)
- Run C integrated validation
- All 181 tests pass
- No internal leakage detected
- Reconstructed bill sections present in all bill reports
- Golden text snapshot tests connected
- PDF quality verified
- Git hygiene clean
- Final verdict: GO
@Kingslayer-bot
Kingslayer-bot merged commit d51a4fe into main Jul 8, 2026
2 checks 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.

2 participants