Design explicit acceptance and review of changed skill content
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
Direzione di ricerca
Inizia leggendo packages/intent/src/core/source-policy.ts e packages/intent/src/core/intent-core.ts, quindi esamina le decisioni correlate nelle issue #204, #220, #221, #222 e #229. Definisci il flusso di revisione e accettazione, i confini dell’identità, la diagnostica, le protezioni dell’autorità e una porzione di implementazione con un comportamento di rifiuto visibile e test; il lavoro è completo quando i criteri di accettazione sono specificati senza aggiungere plumbing limitato allo storage.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Problem
Current permission selects a source or skill name, not a reviewed content snapshot. An allowed skill can change without a version change and load immediately. This is documented name-based trust behavior, not a bypass of an existing content lock.
Evidence
Replacing an allowed SKILL.md body while preserving package name and version changed the content returned by loadIntentSkill on the next call.
Sources: permission matching, live content read. #204 previously explored broader accepted-state work; #220 and #221 explicitly exclude byte-level content verification.
Verified against main at 206e987 (@tanstack/intent 0.3.8), Node v24.12.0 on macOS arm64. Tests used installed dependencies and synthetic local fixtures.
Acceptance criteria
- Define the user flow for reviewing, accepting, and refusing changed content before choosing a storage format.
- Define source identity and the acceptance boundary for SKILL.md, referenced resources, scripts, new files, deletions, and path escapes.
- Specify what happens when package version stays unchanged but accepted content changes.
- Distinguish content equality, publisher provenance, and instruction safety in diagnostics and documentation.
- Define who may update acceptance state and what host/filesystem protections are required for a hard authority boundary.
- A proposed implementation slice includes visible review/refusal behavior and tests, not storage-only plumbing.
Scope and priority
Priority: High. Type: feature design.
Separate design and follow-up release, not a v0.4.0 dependency. Reuse #222 for policy explanations and #229 for command-authority work. Do not widen #221 into content drift. No hosted registry, Git sources, native delivery synchronization, or claim that a hash proves safety.
- Lingua principale
- TypeScript
- Stelle
- 331
- Fork
- 22
- Merge medio
- 12h 17m
- PR unite (30g)
- 51
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di TanStack/intent
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 45/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 45/100
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
Tutte le issue di TanStack/intent
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
bug v2
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
modelcontextprotocol/inspector#2458 · 1 commento ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100
railmapgen/rmp-gallery#4068 ·
-
Mend: dependency security vulnerability status: needs triage 🕵️♀️
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
carbon-design-system/ibm-products#9907 ·