fix(interactions): resolve alias slug trong api.user-interactions #47

Activity

1. clark-cant commented on Sep 22, 2026

   

   clark-cant

   on Sep 22, 2026

   More actions

   Xác minh độc lập trên main (sau #50, repo pushed 2026-09-22T10:50Z)

   Core claim đúng: api.user-interactions.ts tra thẳng inArray(skills.slug, slugs) nên chỉ khớp canonical slug; client gửi slug cũ nhận overlay rỗng (favorites: [], votes: {}) mà không có lỗi nào.

   Nhưng phạm vi thực tế rộng hơn issue mô tả — đây không phải lỗ hổng của riêng api.user-interactions.ts. Kiểm tra main, các route sau cũng tra thẳng eq(skills.slug, …) và bỏ qua skill_aliases:

   Route	Điểm tra	Hành vi với slug cũ
   api.user-interactions.ts	loader — inArray(skills.slug, slugs)	overlay rỗng, im lặng
   api.skill-review.ts	loader và action — eq(skills.slug, slug)	404 "Skill not found" (cả đọc lẫn ghi review)
   api.skill-rate.ts	action — eq(skills.slug, slug)	404 "Skill not found"
   api.skill-vote.ts	action — eq(skills.slug, slug)	404 "Skill not found"
   api.usage-report.ts	action — eq(skills.slug, skill_slug)	404 "Skill not found"

   Chỉ api.skill-detail.ts, api.skill-favorite.ts, api.skill-install.ts (qua lib/catalog/skill-detail-data.ts → skill-detail-queries.ts) là đi qua resolveSkillBySlug.

   Hệ quả: hợp đồng "mọi slug cũ vẫn resolve" mà #43 thiết lập hiện chưa đúng ở 5 route, không phải 1. Và fail mode còn nặng hơn phần "Mức độ" issue ghi: rate/vote/review với slug cũ trả 404 cứng, không chỉ mất overlay như user-interactions.

   Đề xuất

   • Thay vì vá lẻ từng route, thêm một helper dùng chung — ví dụ resolveSkillIds(db, slugs: string[]) gộp canonical (skills.slug) ∪ alias (skill_aliases.slug → skill_id) — rồi cho cả 5 điểm tra dùng nó. resolveSkillBySlug hiện chỉ trả 1 row; các route này cần bản nhiều slug (batch).
   • Thêm test cho mỗi route: slug cũ trỏ tới skill có favorite/vote/review/usage vẫn trả đúng thay vì rỗng/404.

   Nhãn

   Gắn bug (không phải enhancement): nhất quán với nghĩa "Something isn't working" — endpoint trả 404/rỗng cho slug hợp lệ cũ. Giữ issue open.
2. clark-cant

   added

   bugSomething isn't workingSomething isn't working

   on Sep 22, 2026
3. mrgoonie

   added

   in progressBeing worked onBeing worked on

   on Oct 19, 2026
4. mrgoonie

   self-assigned this

   on Oct 19, 2026
5. mrgoonie

   added

   in reviewPR under reviewPR under review

   and removed

   in progressBeing worked onBeing worked on

   on Oct 19, 2026
6. mrgoonie

   added a commit that references this issue on Oct 20, 2026

   fix(web): resolve alias slugs on interactions, review, rate, vote and…

   ...

   This issue will close once commit 1d5c887 is merged into the 'main' branch.

   1d5c887
7. mrgoonie

   closed this as completedin #96on Oct 20, 2026
8. mrgoonie commented on Oct 20, 2026

   

   mrgoonie

   on Oct 20, 2026

   ContributorAuthor

   More actions

   Shipped: fixed by #96 (squash fbd0333, merged 2026-10-20 after 6 isolated review rounds). Deployed to production skillx-web (worker version 27d34bce at fbd0333; current 7d29ff0a at 79f98fb). Smoke 2026-10-20: /, /api/search and registry keys 200.
9. mrgoonie

   added

   shippedMerged/deployedMerged/deployed

   and removed

   in reviewPR under reviewPR under review

   on Oct 20, 2026
10. LoveDoLove

    added a commit that references this issue on Oct 20, 2026

    fix: alias slugs, .pi untracking, release scan tuning, API key settin…

    ...

    This issue will close once commit fbd0333 is merged into the 'main' branch.

    fbd0333