Skip to content

chore: remove unused prior-art-template from investigation-reporting#50

Merged
sonesuke merged 19 commits intomainfrom
chore/remove-unused-prior-art-template
Apr 4, 2026
Merged

chore: remove unused prior-art-template from investigation-reporting#50
sonesuke merged 19 commits intomainfrom
chore/remove-unused-prior-art-template

Conversation

@sonesuke
Copy link
Copy Markdown
Owner

@sonesuke sonesuke commented Apr 4, 2026

Summary

  • Remove assets/prior-art-template.md as it is already integrated into specific-patent-report-template.md
  • Remove corresponding reference from SKILL.md

Test plan

  • claude plugin validate . passes

🤖 Generated with Claude Code

web-flow and others added 19 commits April 4, 2026 08:11
Already integrated into specific-patent-report-template.md.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…emplate

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Merge references/instructions.md into SKILL.md for single-file clarity.
Add user_invocable: false and context: fork since this is an internal
skill invoked by other skills via the Skill tool.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Merge references/instructions.md into SKILL.md for single-file clarity.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace tool-param checks with skill-invoked for patent-search, and add
copy_to_output for generated files.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
file-contains and message-contains were not matching correctly.
Switch to log-contains for reliable text verification.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Write specific patent report to <patent_id>.md instead of text output
- Verify legal-checking skill is invoked after report generation
- Add copy_to_output for generated report file in test

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace file parsing with database queries via investigation-fetching skill
for both overall-progress and specific-patent report modes. Add legal-checking
invocation step after report generation.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Report now reflects investigation workflow progress. Only completed phases
(screening, evaluation, claim analysis, prior art) show data; incomplete
phases display "Pending" with remaining actions.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Verify report shows Pending for incomplete phases (screening done,
no evaluation/claim/prior-art) and includes relevant screening result.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…rt template

Move "Get screening progress statistics" from investigation-preparing to
investigation-fetching where it belongs as a read operation. Add Count-type
operations to investigation-fetching (screening, claim analysis, prior art).

Redesign investigation-report-template.md:
- Merge redundant Overview/Screening Summary into single Screening section
- Replace Investigation Progress with Claim Analysis (All/Limited/Not Limited)
- Add Prior Art section (All/Resolved/Open/Pending)
- Remove Quality Gates from template (kept in instructions only)

Update overall-progress-report.md to use investigation-fetching Count APIs
instead of raw Execute SQL calls.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Model was ignoring the template and generating its own structure with
extra sections (Evaluation, Overview) and non-standard metric names.

Root cause: "DO NOT read files from references/instructions/" warning was
blocking the model from reading assets/ templates too.

Changes:
- Clarify that assets/ templates CAN be read (only references/instructions/ is restricted)
- Add Step 0 (MANDATORY): read template before any other action
- Add explicit metric name requirements (Targets, Screened, etc.)
- Remove outdated Examples/Troubleshooting sections from SKILL.md
- Update test checks to verify template section names

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Replace "Verdict" with neutral terms in specific-patent-report-template.md:
- Element Analysis table: Verdict → Similarity
- Claim Analysis summary: Verdict → Summary
- Prior Art conclusion: Verdict → Result

Add functional-specific-patent-with-prior-art test that seeds complete
investigation data including prior_arts and prior_art_elements.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@sonesuke sonesuke merged commit 81d2f65 into main Apr 4, 2026
3 checks passed
@sonesuke sonesuke deleted the chore/remove-unused-prior-art-template branch April 4, 2026 11:50
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