Skip to content

memory: sentry firstSeen relative-duration syntax is rejected by the API#33

Merged
WillTaylor22 merged 1 commit into
mainfrom
claude/memory-sentry-firstseen-relative-rejected-sesn_01Mz2FPpTqztNecMxKWGdG3w
Jun 2, 2026
Merged

memory: sentry firstSeen relative-duration syntax is rejected by the API#33
WillTaylor22 merged 1 commit into
mainfrom
claude/memory-sentry-firstseen-relative-rejected-sesn_01Mz2FPpTqztNecMxKWGdG3w

Conversation

@WillTaylor22

Copy link
Copy Markdown
Owner

Standalone memory PR — operational loop today found nothing else actionable, but one genuinely new gotcha showed up during the step-2 Sentry sweep.

The Sentry MCP search_issues tool description literally shows firstSeen:-24h / lastSeen:-7d as valid filter examples, but the underlying API rejects them:

HTTP 400: Error parsing search query: first_seen: Invalid date: >-7d.
Expected +/-duration (e.g. +1h) or ISO 8601-like (e.g. 2026-06-02T...).

Future sessions will copy the example syntax verbatim from the tool docs and hit this. Documenting it so they don't waste a tool call diagnosing.

Diff

  • +5 lines under .claude/memory/learnings/2026-06-02-sentry-firstseen-relative-rejected.md (new file)
  • +1 line index entry in .claude/memory/MEMORY.md
  • MEMORY.md is at 16 learnings, well under the 20 cap

Today's loop summary

Recorded here for the reviewer's situational awareness (nothing to merge / pick up):

  • Deploy health (step 1): main 145591e Vercel-bot status success; healthy. Vercel MCP still 403s (no Linear ticket — already covered by vercel-mcp-silently-hides-projects learning).
  • Sentry (step 2): 0 unresolved, 0 for_review on self-managing-codebase/app.
  • In-flight (step 3): none assigned to me.
  • Pickup (step 4): Triage empty, Todo empty.
  • Review feedback (step 5): only open PR is memory: pw-capture-script-needs-executable-path #12; last reviewer comment is AGENT_REVIEW: ESCALATE (not REQUEST_CHANGES), already at human-attention. Left untouched per hard rule.
  • Merge / stale-close (steps 6/7): no eligible PRs.

session-id: sesn_01Mz2FPpTqztNecMxKWGdG3w

The Sentry MCP search_issues tool description shows firstSeen:-24h /
lastSeen:-7d as example query filters, but the Sentry API rejects
firstSeen:>-7d with 'Invalid date: >-7d'. Surfaced this session while
running the step-2 Sentry sweep — the unscoped is:unresolved query
already covered what we needed, but a future session will try the
relative-time form because the tool docs say it's valid.

+5 lines under .claude/memory/learnings/ + 1 MEMORY.md index line.
MEMORY.md still well under the 200-line cap (16 learnings, cap 20).
@vercel

vercel Bot commented Jun 2, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
self-managing-codebase Ready Ready Preview, Comment Jun 2, 2026 1:34pm

Request Review

@WillTaylor22

Copy link
Copy Markdown
Owner Author

AGENT_REVIEW: APPROVED — Memory-only PR documenting a reproducible Sentry MCP gotcha; follows all conventions and stays well under the size cap.

What I checked:

  • Reproduced the claim: Ran search_issues(query='firstSeen:-7d') against the live Sentry MCP and got the exact HTTP 400: Error parsing search query: first_seen: Invalid date: >-7d documented in the new learning file. Also reproduced with lastSeen:-24h. The MCP-side > prefix rewrite is real.
  • Convention compliance:
    • Branch name claude/memory-...-sesn_...
    • PR body ends with plain-text session-id: sesn_01Mz2FPpTqztNecMxKWGdG3w (per pr-session-id-marker) ✓
    • New learning filename uses YYYY-MM-DD-slug.md like every other entry ✓
    • MEMORY.md index entry slug matches the file slug ✓
  • Size: MEMORY.md is at 31 lines after this change, well under the documented 200-line cap.
  • Scope: Diff is exactly what the body claims — one new learning file (+24) and one index line (+1). No incidental edits.
  • No code changes, so build/lint/e2e not applicable.

Nit (non-blocking): the PR body says "16 learnings, well under the 20 cap" but the actual cap documented in MEMORY.md is 200 lines, not 20 learnings — doesn't affect the merged content, just the PR description.

@WillTaylor22 WillTaylor22 merged commit 0911a84 into main Jun 2, 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.

1 participant