[integrations] Fix smart-ingest non-dry-run commit bugs

[symy1610]

Summary

Fixes the two bugs that block every real (non-dry-run) smart-ingest write, tracked in #379 and #468.

#379 — UUID/bigint mismatch on upsert_thought return.
upsert_thought's v_id is UUID (schemas/enhanced-thoughts/schema.sql), serialized as a JSON string. extractThoughtId() in integrations/smart-ingest/index.ts only accepted numbers (Number.isFinite), so every successful write was read back as "no thought_id" and the item marked failed — even though the row had already landed in thoughts. ingestion_items.matched_thought_id / result_thought_id were also declared bigint, so they couldn't hold a UUID even if the client accepted one, and append_thought_evidence(bigint, ...) compared a bigint parameter against thoughts.id (uuid), which can't succeed.

Fix:

• extractThoughtId() accepts UUID strings (numeric ids still accepted, for bigint-id forks).
• ingestion_items.matched_thought_id / result_thought_id are now uuid columns, with an idempotent migration for existing installs (safe — the bug meant these were always NULL in practice).
• append_thought_evidence now takes p_thought_id uuid; the old bigint overload is dropped rather than left dangling.

#468 — OpenRouter extraction fails on instruction-shaped content.
response_format: { type: "json_object" } is an OpenAI-shaped parameter; the default OpenRouter model (an Anthropic model) silently ignores it, so it can still respond with prose instead of JSON — reproduced live as an HTTP 500 "OpenRouter returned invalid JSON".

Fix: per the issue's suggested fix #1, retry once with an explicit "return only JSON" correction before failing. Left a comment on response_format explaining it isn't a reliable guard on its own.

What it requires

• Same as smart-ingest today: Supabase + pgvector, enhanced-thoughts schema applied (for upsert_thought), OpenRouter/OpenAI/Anthropic key.
• The schema change is additive/idempotent — safe to reapply schemas/smart-ingest/schema.sql on an existing install.

Testing

Repro'd both failures against a live OB1 deployment before this fix:

• smart-ingest & enhanced-mcp: writes silently orphan — UUID/bigint mismatch on upsert_thought return #379: real (non-dry_run) POST → HTTP 200, added_count: 0, failed_count: 1; ingestion_items.error_message = 'upsert_thought returned no thought_id'.
• response_format json_object is a no-op on the default OpenRouter model #468: real POST → HTTP 500 {"error":"Extraction failed","reason":"extraction_failed"}; ingestion_jobs.error_message = 'OpenRouter non-transient failure (no fallback): OpenRouter returned invalid JSON'.

For this change itself: deno check and deno lint pass on integrations/smart-ingest (the one pre-existing lint warning in _shared/helpers.ts and the deno fmt diff on index.ts both predate this PR — confirmed present on origin/main before these edits, unrelated to this change). I was not able to redeploy and re-verify end-to-end against a live Supabase instance as part of this PR (no test project available in this environment) — flagging that per CONTRIBUTING.md's testing guidance, and happy to verify against a real instance if a maintainer can point me at one, or if the original reporter (who offered to send a PR in #379) wants to confirm independently.

Closes #379, and fixes the OpenRouter side of #468 for smart-ingest specifically (the issue also covers the general "no guard on non-JSON responses" observation, which this PR's retry addresses for this call site).

[github-actions]

Hey @sydney-moutia — welcome to Open Brain Source! 👋

Thanks for submitting your first PR. The automated review will run shortly and check things like metadata, folder structure, and README completeness. If anything needs fixing, the review comment will tell you exactly what.

Once the automated checks pass, a human admin will review for quality and clarity. Expect a response within a few days.

If you have questions, check out CONTRIBUTING.md or open an issue.