fix(memory): handle JSON-string tags from LLM serialization (v1.2.1)

[konard]

Summary

Fixes #51

Root Cause

The error tags: must be array occurs because LLMs sometimes serialize array arguments as a JSON-encoded string instead of a proper JSON array:

// What the plugin expects:
{ "tags": ["rules", "github"] }

// What LLMs sometimes send:
{ "tags": "[\"rules\", \"github\"]" }

The external validation layer in @mariozechner/pi-ai checks the JSON Schema before calling execute, so the plugin never had a chance to handle this gracefully. The issue is intermittent because LLMs don't do this consistently — they sometimes pass a real array, sometimes a string.

Fix

Two complementary changes:

1. Schema widened — Each tags parameter schema is changed from type: "array" to oneOf: [array, string], so the validator accepts both forms and lets execute run.

2. coerceToArray() helper added — Normalises a JSON string, plain string, or proper array to a plain JS array before processing. Used in parseTags() (for memory_store / memory_update) and in memory_search tag filter normalisation.

Tests Added

• memory_store — accepts tags as a JSON-encoded string '["rules", "github"]'
• memory_store — accepts tags as a single plain string "work"
• memory_search — accepts tags filter as a JSON-encoded string '["github"]'

All 62 tests pass.

---

This PR was created automatically by the AI issue solver

[konard]

🤖 Solution Draft Log

This log file contains the complete execution trace of the AI solution draft process.

💰 Cost estimation:

• Public pricing estimate: $0.971547
• Calculated by Anthropic: $0.662551 USD
• Difference: $-0.308995 (-31.80%)

🤖 Models used:

• Tool: Anthropic Claude Code
• Requested: sonnet
• Model: Claude Sonnet 4.6 (claude-sonnet-4-6)

📎 Log file uploaded as Gist (943KB)

• View complete solution draft log

---

Now working session is ended, feel free to review and add any feedback on the solution draft.

[konard]

✅ Ready to merge

This pull request is now ready to be merged:

• All CI checks have passed
• No merge conflicts
• No pending changes

---

Monitored by hive-mind with --auto-restart-until-mergeable flag