feat(claude-usage): POST et GET /claude-usage (#48) #52

Merged

maximus merged 2 commits from issue-48-claude-usage-routes into main 2026-10-04 19:29:24 +00:00

Conversation 2 Commits 2 Files changed 9 +1428 -41

maximus commented 2026-10-04 13:16:39 +00:00

Owner

Copy link

Resume

Ajoute POST /claude-usage (ingestion du snapshot de consommation Claude pousse par l'agent du ThinkPad) et GET /claude-usage (lecture pour le dashboard /admin), contrat schemaVersion: 1.

• Gate : checkAuth() lit le token attendu dans une table TOKEN_KINDS (read -> HEALTH_TOKEN, ingest -> HOSTS_INGEST_TOKEN, claude-ingest -> CLAUDE_USAGE_INGEST_TOKEN). Toujours un seul point de controle, un seul appel tokenMatches(), un seul return null. Chaque token d'ingestion absent -> 503 nommant sa propre variable. Un tokenKind inconnu dans ROUTES fait echouer le chargement du module.
• readBody(req, res, maxBytes) : 4 096 pour /hosts/<id>, 65 536 pour /claude-usage, compte sur les octets recus.
• claude-usage.js (nouveau, pur, ajoute au COPY) : sanitizeClaudeUsage() + bornes exportees. Liste blanche ; valeur invalide jetee (cle omise), jamais forcee ; seuls « pas un objet » et schemaVersion !== 1 refusent le corps (400).
• POST : 400 JSON invalide, sanitize, receivedAt serveur, ecriture atomique (temporaire suffixe du pid, mode 600, rename), 204.
• GET : fichier re-sanitise + ageSeconds, stale (> CLAUDE_USAGE_STALE_SECONDS, defaut 2700), staleAfterSeconds ; {status:"no_data"} si absent ou illisible.
• Rejets 401/503 journalises par le gate comme les autres routes.
• Docs : README.md, CLAUDE.md (endpoints, routage/auth, config, montage, tests, gotchas), .env.example.

Decisions (autopilot)

• [MEDIUM] Une ligne de __tests__/auth.test.js modifiee — ecart au critere « suite existante sans modification autre que la liste COPY ». L'epingle de source de checkAuth() autorisait exactement les egalites header === "string" et route.tokenKind === "ingest". Le refactor en table demande par l'issue supprime la seconde par construction ; la garder aurait impose une comparaison morte dans le gate. L'allowlist passe a ['header === "string"'] seule : l'epingle ne fait que se resserrer. Toutes les autres epingles (un tokenMatches, un return null, espion timingSafeEqual, hachage) sont intactes.
• [MEDIUM] __tests__/writes.test.js etendu (exige par CLAUDE.md pour tout nouveau chemin d'ecriture) : violations(writes, allowedDir = HOSTS_DIR) (appels existants inchanges), fixture + token Claude, /claude-usage dans READ_ROUTES, deux cas : POST /claude-usage n'ecrit que dans le dossier de CLAUDE_USAGE_PATH ; ses rejets 401/400/413 et la lecture d'un fichier corrompu n'ecrivent rien. Aucune assertion existante affaiblie.
• [HIGH] __tests__/agent.test.js non modifie : son test interdit COPY . . et l'agent mais ne fige pas la liste ; un pin positif « le COPY embarque claude-usage.js » est dans le nouveau fichier de test.
• [MEDIUM] Bornes de liste tronquees, pas refusees : days garde les 31 plus recents (ordre du contrat : plus ancien -> aujourd'hui), models/items/projects gardent les N premiers.
• [HIGH] percent doit etre un entier 0-100 (lecture litterale de « nombres Number.isSafeInteger ») : l'agent (issue suivante) doit arrondir, sinon le pourcentage est jete.
• [HIGH] Pas de balayage des .tmp orphelins pour /data/claude-usage : inoffensifs (GET ne lit que le fichier final), et l'ajout toucherait le bloc require.main epingle. Documente dans les gotchas.
• test-curl.sh non modifie (45 cas figes dans ci.test.js).

Tests

• npm test : 500 cas (377 avant ; +119 claude-usage.test.js, +3 writes.test.js, +1 ci.test.js par l'epingle require en colonne 0 sur le nouveau module)
• bash test-curl.sh : 45/45
• sh scripts/boot-from-copy.sh : PASS

Actions requises de Max

1. Coolify, env var CLAUDE_USAGE_INGEST_TOKEN (secret fort, distinct des deux autres) en is_runtime=true, is_buildtime=false. Sans elle, POST /claude-usage repond 503 (fail-closed).
2. Coolify, Persistent Storage : Volume Mount avec Source Path /data/claude-usage (hote) -> /data/claude-usage (conteneur), lecture-ecriture, sans :ro. Monter le dossier, jamais le fichier.
3. Sur le VPS avant le redeploy : sudo mkdir -p /data/claude-usage && sudo chown 1000:1000 /data/claude-usage (sinon chaque POST repond 500).
4. Optionnel : CLAUDE_USAGE_STALE_SECONDS (defaut 2700) et CLAUDE_USAGE_PATH (defaut /data/claude-usage/claude-usage.json).
5. Redeploy manuel (pas d'auto-deploy), puis verifier docker inspect (RW=true sur /data/claude-usage) et GET /claude-usage -> {"status":"no_data"} avant le premier push de l'agent.

Generated autonomously by /autopilot run of 2026-10-04

🤖 Generated with Claude Code

## Resume Ajoute `POST /claude-usage` (ingestion du snapshot de consommation Claude pousse par l'agent du ThinkPad) et `GET /claude-usage` (lecture pour le dashboard `/admin`), contrat `schemaVersion: 1`. - **Gate** : `checkAuth()` lit le token attendu dans une table `TOKEN_KINDS` (`read` -> `HEALTH_TOKEN`, `ingest` -> `HOSTS_INGEST_TOKEN`, `claude-ingest` -> `CLAUDE_USAGE_INGEST_TOKEN`). Toujours un seul point de controle, un seul appel `tokenMatches()`, un seul `return null`. Chaque token d'ingestion absent -> 503 nommant sa propre variable. Un `tokenKind` inconnu dans `ROUTES` fait echouer le chargement du module. - **`readBody(req, res, maxBytes)`** : 4 096 pour `/hosts/<id>`, 65 536 pour `/claude-usage`, compte sur les octets recus. - **`claude-usage.js`** (nouveau, pur, ajoute au `COPY`) : `sanitizeClaudeUsage()` + bornes exportees. Liste blanche ; valeur invalide jetee (cle omise), jamais forcee ; seuls « pas un objet » et `schemaVersion !== 1` refusent le corps (400). - **POST** : 400 JSON invalide, sanitize, `receivedAt` serveur, ecriture atomique (temporaire suffixe du pid, mode 600, `rename`), 204. - **GET** : fichier re-sanitise + `ageSeconds`, `stale` (`> CLAUDE_USAGE_STALE_SECONDS`, defaut 2700), `staleAfterSeconds` ; `{status:"no_data"}` si absent ou illisible. - Rejets 401/503 journalises par le gate comme les autres routes. - Docs : `README.md`, `CLAUDE.md` (endpoints, routage/auth, config, montage, tests, gotchas), `.env.example`. ## Decisions (autopilot) - **[MEDIUM] Une ligne de `__tests__/auth.test.js` modifiee** — ecart au critere « suite existante sans modification autre que la liste COPY ». L'epingle de source de `checkAuth()` autorisait exactement les egalites `header === "string"` et `route.tokenKind === "ingest"`. Le refactor en table demande par l'issue supprime la seconde par construction ; la garder aurait impose une comparaison morte dans le gate. L'allowlist passe a `['header === "string"']` seule : l'epingle ne fait que se resserrer. Toutes les autres epingles (un `tokenMatches`, un `return null`, espion `timingSafeEqual`, hachage) sont intactes. - **[MEDIUM] `__tests__/writes.test.js` etendu** (exige par CLAUDE.md pour tout nouveau chemin d'ecriture) : `violations(writes, allowedDir = HOSTS_DIR)` (appels existants inchanges), fixture + token Claude, `/claude-usage` dans `READ_ROUTES`, deux cas : `POST /claude-usage` n'ecrit que dans le dossier de `CLAUDE_USAGE_PATH` ; ses rejets 401/400/413 et la lecture d'un fichier corrompu n'ecrivent rien. Aucune assertion existante affaiblie. - **[HIGH] `__tests__/agent.test.js` non modifie** : son test interdit `COPY . .` et l'agent mais ne fige pas la liste ; un pin positif « le COPY embarque `claude-usage.js` » est dans le nouveau fichier de test. - **[MEDIUM] Bornes de liste tronquees, pas refusees** : `days` garde les 31 plus recents (ordre du contrat : plus ancien -> aujourd'hui), `models`/`items`/`projects` gardent les N premiers. - **[HIGH] `percent` doit etre un entier 0-100** (lecture litterale de « nombres `Number.isSafeInteger` ») : **l'agent (issue suivante) doit arrondir**, sinon le pourcentage est jete. - **[HIGH] Pas de balayage des `.tmp` orphelins pour `/data/claude-usage`** : inoffensifs (GET ne lit que le fichier final), et l'ajout toucherait le bloc `require.main` epingle. Documente dans les gotchas. - `test-curl.sh` non modifie (45 cas figes dans `ci.test.js`). ## Tests - `npm test` : **500 cas** (377 avant ; +119 `claude-usage.test.js`, +3 `writes.test.js`, +1 `ci.test.js` par l'epingle `require` en colonne 0 sur le nouveau module) - `bash test-curl.sh` : 45/45 - `sh scripts/boot-from-copy.sh` : PASS ## Actions requises de Max 1. **Coolify, env var** `CLAUDE_USAGE_INGEST_TOKEN` (secret fort, distinct des deux autres) en `is_runtime=true, is_buildtime=false`. Sans elle, `POST /claude-usage` repond 503 (fail-closed). 2. **Coolify, Persistent Storage** : Volume Mount avec Source Path `/data/claude-usage` (hote) -> `/data/claude-usage` (conteneur), **lecture-ecriture**, sans `:ro`. Monter le dossier, jamais le fichier. 3. **Sur le VPS** avant le redeploy : `sudo mkdir -p /data/claude-usage && sudo chown 1000:1000 /data/claude-usage` (sinon chaque POST repond 500). 4. Optionnel : `CLAUDE_USAGE_STALE_SECONDS` (defaut 2700) et `CLAUDE_USAGE_PATH` (defaut `/data/claude-usage/claude-usage.json`). 5. Redeploy manuel (pas d'auto-deploy), puis verifier `docker inspect` (`RW=true` sur `/data/claude-usage`) et `GET /claude-usage` -> `{"status":"no_data"}` avant le premier push de l'agent. Generated autonomously by /autopilot run of 2026-10-04 🤖 Generated with [Claude Code](https://claude.com/claude-code)

maximus added 2 commits 2026-10-04 13:16:39 +00:00

feat(claude-usage): add POST and GET /claude-usage behind a third token kind ... e1224e0df4

- checkAuth() reads the expected token from a TOKEN_KINDS table
  (read -> HEALTH_TOKEN, ingest -> HOSTS_INGEST_TOKEN, claude-ingest ->
  CLAUDE_USAGE_INGEST_TOKEN); each ingestion token fails closed to a 503
  naming its own variable. Still one gate, one tokenMatches() call. A route
  naming an unknown kind fails module load.
- readBody() takes its ceiling per route: 4 KiB for /hosts/<id>, 64 KiB for
  /claude-usage.
- claude-usage.js: pure, exported sanitizeClaudeUsage() for contract
  schemaVersion 1 (whitelist rebuild, invalid values dropped, list bounds).
- POST /claude-usage: 400 on invalid JSON or schemaVersion, server-stamped
  receivedAt, atomic write (pid-suffixed temp, mode 600, rename), 204.
- GET /claude-usage: re-sanitized snapshot with ageSeconds, stale and
  staleAfterSeconds (CLAUDE_USAGE_STALE_SECONDS, default 2700), or no_data.
- Dockerfile COPY ships claude-usage.js.
- auth.test.js: the checkAuth() equality allowlist loses the former
  `route.tokenKind === "ingest"` entry, which the table removes. The pin
  only got stricter.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

test(claude-usage): cover the routes, the sanitizer and the new write surface ... ef8558e90e

- __tests__/claude-usage.test.js (119 cases): sanitizer contract and
  idempotence, dropped values (unknown fields, negative, past 2^53,
  detail outside its regex, status outside the enum, bad dates, model
  keys), list bounds, token isolation both ways, 503 naming each variable,
  413 at 65 537 bytes and /hosts/<id> still 413 at 4 097, atomic mode-600
  write, POST then GET round trip, exact staleness boundary under a frozen
  clock, no_data degradation, rejection logs, COPY line.
- writes.test.js: POST /claude-usage may write only in the directory of
  CLAUDE_USAGE_PATH, its rejections write nothing, GET /claude-usage joins
  the read routes.
- README.md and CLAUDE.md: both routes, the three env vars, the
  /data/claude-usage read-write mount owned by uid 1000, the token table.

Resolves #48

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

maximus added the

autopilot:pending-human

label 2026-10-04 13:16:42 +00:00

maximus commented 2026-10-04 13:17:32 +00:00

Author

Owner

Copy link

Auto-revue (autopilot) — verdict : pret pour revue humaine, un ecart assume

Relecture de git diff main...HEAD critere par critere :

Critere d'acceptation	Etat	Preuve
POST sans token / mauvais token -> 401 ; token postes sur /claude-usage -> 401 ; token Claude sur POST /hosts/<id> -> 401	OK	claude-usage.test.js « POST /claude-usage — authentication » (6 cas + croisement /hosts/thinkpad), plus token Claude refuse sur 4 routes GET
CLAUDE_USAGE_INGEST_TOKEN absent -> 503 nommant la variable ; /hosts garde son 503 HOSTS_INGEST_TOKEN	OK	bloc « fail-closed — each ingestion token names its own variable » ; hosts.test.js inchange et vert
65 537 octets -> 413 ; /hosts/<id> 413 a 4 097	OK	bloc « the 64 KiB ceiling » (65 536 accepte, 65 537 refuse, hosts 4 097 refuse)
JSON invalide -> 400 ; champ inconnu, negatif, > 2^53, detail hors regex, status hors enum jetes	OK	blocs sanitizer « dropped, not coerced »
POST puis GET -> snapshot identique + receivedAt, ageSeconds, stale	OK	« POST then GET » + bornes 2699/2700/2701 sous horloge figee
GET sans token -> 401 ; fichier absent -> no_data	OK	« GET /claude-usage — authentication », « no file »
Suite existante sans modification autre que la liste COPY	Ecart	une assertion de auth.test.js resserree (allowlist d'egalites de checkAuth() : route.tokenKind === "ingest" disparait avec la table). Cas ajoutes a writes.test.js (exige par CLAUDE.md). Voir decisions dans la description.

Points verifies en plus :

• Gate : un seul tokenMatches(), un seul return null, aucune egalite sur un secret ; espion timingSafeEqual vert pour la nouvelle route ; ordre routage -> auth intact (404 avant 401 sur PUT/DELETE/sous-chemins).
• Tous les require en colonne 0 (epingle ci.test.js passe aussi sur claude-usage.js), claude-usage.js dans le COPY, boot-from-copy.sh PASS.
• Ecriture atomique en mode 600, aucun .tmp laisse ; 500 propre si le dossier est inutilisable.
• Stabilite : writes.test.js + claude-usage.test.js relances 5 fois, 133/133 a chaque fois.

Points a l'attention de Max :

• percent doit etre entier : l'agent doit arrondir.
• Pas de balayage des .tmp orphelins dans /data/claude-usage (documente).
• Rien n'empeche techniquement de poser la meme valeur pour CLAUDE_USAGE_INGEST_TOKEN et HOSTS_INGEST_TOKEN : l'isolation suppose deux secrets distincts.

Validation : npm test 500/500, bash test-curl.sh 45/45, sh scripts/boot-from-copy.sh PASS.

## Auto-revue (autopilot) — verdict : pret pour revue humaine, un ecart assume Relecture de `git diff main...HEAD` critere par critere : | Critere d'acceptation | Etat | Preuve | |---|---|---| | POST sans token / mauvais token -> 401 ; token postes sur `/claude-usage` -> 401 ; token Claude sur `POST /hosts/<id>` -> 401 | OK | `claude-usage.test.js` « POST /claude-usage — authentication » (6 cas + croisement `/hosts/thinkpad`), plus token Claude refuse sur 4 routes GET | | `CLAUDE_USAGE_INGEST_TOKEN` absent -> 503 nommant la variable ; `/hosts` garde son 503 `HOSTS_INGEST_TOKEN` | OK | bloc « fail-closed — each ingestion token names its own variable » ; `hosts.test.js` inchange et vert | | 65 537 octets -> 413 ; `/hosts/<id>` 413 a 4 097 | OK | bloc « the 64 KiB ceiling » (65 536 accepte, 65 537 refuse, hosts 4 097 refuse) | | JSON invalide -> 400 ; champ inconnu, negatif, > 2^53, `detail` hors regex, `status` hors enum jetes | OK | blocs sanitizer « dropped, not coerced » | | POST puis GET -> snapshot identique + `receivedAt`, `ageSeconds`, `stale` | OK | « POST then GET » + bornes 2699/2700/2701 sous horloge figee | | GET sans token -> 401 ; fichier absent -> `no_data` | OK | « GET /claude-usage — authentication », « no file » | | Suite existante sans modification autre que la liste COPY | **Ecart** | une assertion de `auth.test.js` resserree (allowlist d'egalites de `checkAuth()` : `route.tokenKind === "ingest"` disparait avec la table). Cas ajoutes a `writes.test.js` (exige par CLAUDE.md). Voir decisions dans la description. | Points verifies en plus : - Gate : un seul `tokenMatches()`, un seul `return null`, aucune egalite sur un secret ; espion `timingSafeEqual` vert pour la nouvelle route ; ordre routage -> auth intact (404 avant 401 sur PUT/DELETE/sous-chemins). - Tous les `require` en colonne 0 (epingle `ci.test.js` passe aussi sur `claude-usage.js`), `claude-usage.js` dans le `COPY`, `boot-from-copy.sh` PASS. - Ecriture atomique en mode 600, aucun `.tmp` laisse ; 500 propre si le dossier est inutilisable. - Stabilite : `writes.test.js` + `claude-usage.test.js` relances 5 fois, 133/133 a chaque fois. Points a l'attention de Max : - `percent` doit etre entier : l'agent doit arrondir. - Pas de balayage des `.tmp` orphelins dans `/data/claude-usage` (documente). - Rien n'empeche techniquement de poser la meme valeur pour `CLAUDE_USAGE_INGEST_TOKEN` et `HOSTS_INGEST_TOKEN` : l'isolation suppose deux secrets distincts. Validation : `npm test` 500/500, `bash test-curl.sh` 45/45, `sh scripts/boot-from-copy.sh` PASS.

maximus referenced this pull request 2026-10-04 13:18:24 +00:00

feat(agent): aggregate Claude usage from local transcripts (#49) #51

maximus referenced this pull request 2026-10-04 13:29:15 +00:00

feat(agent): claude-usage agent — plan limits, local cache, push (#50) #53

maximus commented 2026-10-04 14:49:24 +00:00

Author

Owner

Copy link

Verdict : APPROVE

Resume : les deux routes /claude-usage passent par le gate unique (table TOKEN_KINDS, un seul tokenMatches(), un seul return null), le sanitizer par liste blanche est idempotent et la nouvelle surface d'ecriture entre dans writes.test.js. Suite, passe socket et boot check rejoues sur la tete ef8558e, plus 11 mutations du relecteur.

Execute (copie hors depot de ef8558e)

• npm test : 500/500 ; bash test-curl.sh : 45/45 ; sh scripts/boot-from-copy.sh : PASS (ports 3097-3099 libres avant lancement)
• CI de la tete : CI / verification (pull_request) = success

Mutations

Mutation	Tests rouges
POST /claude-usage sur tokenKind: "ingest" (token partage avec /hosts)	16
plafond 4 Kio au lieu de 64 Kio	1
ecriture directe sur le fichier final (non atomique)	1
GET sans re-sanitisation du fichier	1
stale en >= au lieu de >	1
days garde les 31 premiers au lieu des 31 derniers	1
receivedAt lu sans controle	2
compteurs fractionnaires acceptes (isFinite)	8
temporaire en mode 644	1
ecriture hors du dossier de CLAUDE_USAGE_PATH	1 (writes.test.js)
suppression du unlinkSync(tmpPath) de nettoyage	0

Suggestions (non bloquantes)

1. __tests__/claude-usage.test.js:739 — le cas « 500 when the snapshot cannot be written, and no temp file is left » echoue au mkdirSync (parent = fichier), avant la creation du temporaire : le nettoyage du catch de writeClaudeUsage() n'est jamais emprunte, et le cas n'affirme pas l'absence de .tmp. La mutation qui retire unlinkSync(tmpPath) reste verte. Pour l'exercer : faire de CLAUDE_USAGE_PATH un dossier existant non vide (le writeFileSync du temporaire passe, le renameSync echoue), puis exiger 500 et aucun .claude-usage.json.*.tmp dans le parent. Ou retirer « no temp file is left » du titre.
2. index.js readClaudeUsage() — le receivedAt relu est valide par new Date(...), permissif ("1" passe et repart tel quel vers le dashboard). Meme comportement que readHostSnapshot(), et seul le serveur ecrit ce champ : passer par un cleanTimestamp exporte de claude-usage.js alignerait la relecture sur la liste blanche du reste du snapshot. A traiter dans une issue de suivi commune aux deux lecteurs plutot qu'ici.
3. Fermeture : Resolves #48 vit dans le message du commit ef8558e, pas dans le corps de la PR (timeline actuelle : ref_action: none). L'issue se fermera quand ce commit atteindra main. Si #48 doit rester ouverte jusqu'aux actions Coolify (token, montage, chown), reformuler avant le merge.

Les ecarts declares (une ligne d'auth.test.js qui ne fait que resserrer l'allowlist, extension de writes.test.js) sont justifies et verifies.

## Verdict : APPROVE **Resume** : les deux routes `/claude-usage` passent par le gate unique (table `TOKEN_KINDS`, un seul `tokenMatches()`, un seul `return null`), le sanitizer par liste blanche est idempotent et la nouvelle surface d'ecriture entre dans `writes.test.js`. Suite, passe socket et boot check rejoues sur la tete `ef8558e`, plus 11 mutations du relecteur. ### Execute (copie hors depot de `ef8558e`) - `npm test` : 500/500 ; `bash test-curl.sh` : 45/45 ; `sh scripts/boot-from-copy.sh` : PASS (ports 3097-3099 libres avant lancement) - CI de la tete : `CI / verification (pull_request)` = success ### Mutations | Mutation | Tests rouges | |---|---| | `POST /claude-usage` sur `tokenKind: "ingest"` (token partage avec `/hosts`) | 16 | | plafond 4 Kio au lieu de 64 Kio | 1 | | ecriture directe sur le fichier final (non atomique) | 1 | | GET sans re-sanitisation du fichier | 1 | | `stale` en `>=` au lieu de `>` | 1 | | `days` garde les 31 premiers au lieu des 31 derniers | 1 | | `receivedAt` lu sans controle | 2 | | compteurs fractionnaires acceptes (`isFinite`) | 8 | | temporaire en mode 644 | 1 | | ecriture hors du dossier de `CLAUDE_USAGE_PATH` | 1 (`writes.test.js`) | | **suppression du `unlinkSync(tmpPath)` de nettoyage** | **0** | ### Suggestions (non bloquantes) 1. `__tests__/claude-usage.test.js:739` — le cas « 500 when the snapshot cannot be written, and no temp file is left » echoue au `mkdirSync` (parent = fichier), avant la creation du temporaire : le nettoyage du `catch` de `writeClaudeUsage()` n'est jamais emprunte, et le cas n'affirme pas l'absence de `.tmp`. La mutation qui retire `unlinkSync(tmpPath)` reste verte. Pour l'exercer : faire de `CLAUDE_USAGE_PATH` un dossier existant non vide (le `writeFileSync` du temporaire passe, le `renameSync` echoue), puis exiger 500 et aucun `.claude-usage.json.*.tmp` dans le parent. Ou retirer « no temp file is left » du titre. 2. `index.js` `readClaudeUsage()` — le `receivedAt` relu est valide par `new Date(...)`, permissif (`"1"` passe et repart tel quel vers le dashboard). Meme comportement que `readHostSnapshot()`, et seul le serveur ecrit ce champ : passer par un `cleanTimestamp` exporte de `claude-usage.js` alignerait la relecture sur la liste blanche du reste du snapshot. A traiter dans une issue de suivi commune aux deux lecteurs plutot qu'ici. 3. Fermeture : `Resolves #48` vit dans le message du commit `ef8558e`, pas dans le corps de la PR (timeline actuelle : `ref_action: none`). L'issue se fermera quand ce commit atteindra `main`. Si #48 doit rester ouverte jusqu'aux actions Coolify (token, montage, `chown`), reformuler avant le merge. Les ecarts declares (une ligne d'`auth.test.js` qui ne fait que resserrer l'allowlist, extension de `writes.test.js`) sont justifies et verifies.

maximus merged commit ef8558e90e into main 2026-10-04 19:29:24 +00:00

maximus referenced this pull request 2026-10-04 19:30:24 +00:00

claude-usage : constats non bloquants des revues de la pile #52/#51/#53 #54

maximus referenced this pull request 2026-10-08 00:30:35 +00:00

fix(claude-usage): review findings of the #52/#51/#53 stack #56

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

autopilot:pending-human

PR produite par /autopilot, attend review humaine (jamais mergee par le run)

  

source:analyste

Créée par le Super-Analyste

  

source:defenseur

Créée par l Escouade Défenseur

  

source:human

Créée par un humain

  

source:medic

Créée par le Medic (auto-détection)

  

status:approved

PR approuvée, prête à merger

  

status:blocked

Bloquée (question, dépendance, clarification)

  

status:in-progress

Un agent (Medic/Dev) travaille dessus

  

status:needs-clarification

Issue bloquee par une ambiguite LOW pendant un run autonome, Max doit trancher

  

status:needs-fix

Reviewer demande des corrections

  

status:ready

Analysée et documentée, prête à prendre

  

status:review

PR créée, en attente du Reviewer

  

status:triage

Nouvelle issue, pas encore analysée

  

type:bug

Correction de bug → Medic

  

type:feature

Nouvelle fonctionnalité → Dev

  

type:infra

Config, CI/CD, déploiement → Dev

  

type:refactor

Refactoring / amélioration technique → Dev

  

type:schema

Modèle de données / migrations → Dev

  

type:security

Issue de sécurité → Medic ou Dev

No labels

autopilot:pending-human

source:analyste

source:defenseur

source:human

source:medic

status:approved

status:blocked

status:in-progress

status:needs-clarification

status:needs-fix

status:ready

status:review

status:triage

type:bug

type:feature

type:infra

type:refactor

type:schema

type:security

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: maximus/vps-health-api#52

No description provided.