user-feedback: cache verification missed non-default purchase situations

Open Beginner friendly
#262 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
88/100
Issue type
Documentation
Clarity
Clearly specified
Activity status
Quiet
Tech stack
supabase
Domain
documentation

Research direction

Open SKILL.md and read Core Principles > 2. Verify your work. Expand the checklist for third-party API caches to cover pagination, date filters, statuses or situations, grouped counts, duplicates, exclusions, and reconciliation against the source UI or an independent aggregate. Done means the guidance prevents a default or partial response from being described as a complete cache load.

Written by the indexing model from the issue text.

Description

What happened

Task: Build and validate a 12-month Supabase cache for purchases imported from the GestãoClick /compras endpoint and use it in a supplier ranking.

Skill said: The Supabase skill requires verification after implementing a fix, but its verification guidance does not explicitly require checking whether a third-party source endpoint has status/situation dimensions whose omission changes the returned dataset.

Expected: Before declaring an imported cache complete, the workflow should compare the source's available situations/statuses with the situations represented in the cache, verify counts by situation, and avoid concluding that a small default response is the complete dataset. In this case, omitting explicit situation enumeration produced only 14 purchases from one default situation; querying all 6 situations produced 1,270 purchases (1,248 valid and 22 cancelled).

Source

File: SKILL.md

Section: Core Principles > 2. Verify your work

Fix suggestion

Expand the verification checklist for caches populated from third-party APIs. Require checking pagination, date filters, hidden/default statuses or situations, counts grouped by source status, duplicates, exclusions, and reconciliation against the source UI or an independent aggregate before describing a cache load as complete.

Dominant language
TypeScript
Stars
2.6k
Forks
208
PR merge metrics
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from supabase/agent-skills

All issues in supabase/agent-skills

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.