koka/sanctuary
1
0
Fork 0
Code Issues 4 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

fix(sanctuary): compter rubis et saphirs aux rangs inventaire 4/5 et 5/5 #10

Merged
koka merged 2 commits from fix/shop-currency-overflow-rows into main 2026-08-27 10:00:08 +00:00
Conversation 1 Commits 2 Files Changed 5
+79 -21
chris commented 2026-08-27 08:28:20 +00:00
Collaborator
Copy Link
Copy Source

Objectif

Le Shop et le marche noir ignoraient des piles de rubis/saphirs selon le rang d'inventaire. Au rang 5/5 (6 rangees), l'overflow n'etait pas compte. Au rang 4/5 (5 rangees), l'avant-derniere ligne (cases vanilla 27-35) restait invisible pour la bourse.

Perimetre

  • Modules touches : sanctuary
  • APIs ou fichiers partages touches : SanctuaryPlayerInventoryView, BlackMarketService.countPlayerItem / removePlayerItem (reutilises par ShopService)
  • Depend d'une autre branche/PR : non
  • Remplace la PR #8 (fermee, non mergee)

Compatibilite inter-mods

  • chaque donnee/regle reste dans son module proprietaire
  • IDs, sauvegardes et protocole inchanges
  • tests adaptes (shopDeliveryTimingSmoke, verifyBlackMarket)
  • version module non incrementee : les pins alpha.200 exigent encore sanctuary 0.0.0-alpha.115 ; a traiter dans le prochain lot

Test plan

  • Inventaire 4/5 (5 rangees), hotbar en bas : gemmes sur l'avant-derniere ligne visibles dans le Shop
  • Inventaire 5/5 (6 rangees), hotbar en bas : 1ere et avant-derniere lignes visibles dans le Shop
  • Un achat / deblocage consomme bien ces piles
## Objectif Le Shop et le marche noir ignoraient des piles de rubis/saphirs selon le rang d'inventaire. Au rang 5/5 (6 rangees), l'overflow n'etait pas compte. Au rang 4/5 (5 rangees), l'avant-derniere ligne (cases vanilla 27-35) restait invisible pour la bourse. ## Perimetre - Modules touches : `sanctuary` - APIs ou fichiers partages touches : `SanctuaryPlayerInventoryView`, `BlackMarketService.countPlayerItem` / `removePlayerItem` (reutilises par `ShopService`) - Depend d'une autre branche/PR : non - Remplace la PR #8 (fermee, non mergee) ## Compatibilite inter-mods - [x] chaque donnee/regle reste dans son module proprietaire - [x] IDs, sauvegardes et protocole inchanges - [x] tests adaptes (`shopDeliveryTimingSmoke`, `verifyBlackMarket`) - [ ] version module non incrementee : les pins alpha.200 exigent encore `sanctuary` `0.0.0-alpha.115` ; a traiter dans le prochain lot ## Test plan - [ ] Inventaire 4/5 (5 rangees), hotbar en bas : gemmes sur l'avant-derniere ligne visibles dans le Shop - [ ] Inventaire 5/5 (6 rangees), hotbar en bas : 1ere et avant-derniere lignes visibles dans le Shop - [ ] Un achat / deblocage consomme bien ces piles
chris added 2 commits 2026-08-27 08:28:20 +00:00
fix(sanctuary): compter rubis et saphirs sur toutes les rangees d'inventaire
Verification 26.2 / check (pull_request) Canceled after 0s
Details
3deeb90ffc 
Co-authored-by: Cursor <cursoragent@cursor.com>
fix(sanctuary): compter aussi les gemmes au rang inventaire 4/5
Verification 26.2 / check (pull_request) Canceled after 0s
Details
3eed6b5a65 
Co-authored-by: Cursor <cursoragent@cursor.com>
chris added 1 commit 2026-08-27 08:28:21 +00:00
fix(sanctuary): compter aussi les gemmes au rang inventaire 4/5
Verification 26.2 / check (pull_request) Canceled after 0s
Details
3eed6b5a65 
Co-authored-by: Cursor <cursoragent@cursor.com>
botsu approved these changes 2026-08-27 09:47:49 +00:00
botsu left a comment
Contributor
Copy Link
Copy Source

Validation ciblée et workspace complet réussis ; combinaison avec alpha.201 vérifiée.

Validation ciblée et workspace complet réussis ; combinaison avec alpha.201 vérifiée.
koka merged commit e7359ac917 into main 2026-08-27 10:00:08 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
botsu
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
chris koka
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: koka/sanctuary#10