Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

configInputToDescriptor drops a remote-only `api` slot (same guard class as the fixed auth drop)

Aperta
  • Qualcosa non va?
#566 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
70/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
typescript
Ambito
cli

Direzione di ricerca

Start by reading the slot helper in src/descriptor/resolve.ts:266-280 and ApiRefSchema in schema.ts:49-58 to understand why remote-only api data is dropped. Then inspect the remote-only auth round-trip case in tests/integration/bos-app-loader.test.ts and add coverage for api variables, plus tests that pin the chosen host/ui behavior. Done means the api entry survives configInputToDescriptor(toConfigInput(descriptor)) and the integration tests pass.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Context

Issue being closed by current work: the descriptor round-trip silently deleted a remote-only auth attachment — configInputToDescriptor gated descriptor.auth on auth?.development (src/descriptor/resolve.ts:283 fixed this PR). The slot() helper for host/ui/api (src/descriptor/resolve.ts:266-280) has the same guard class:

const slot = (key: "host" | "ui" | "api") => {
  const entry = app[key];
  if (!entry?.development) return undefined;   // <- same drop

Impact

A child that overrides api variables/secrets without a local api workspace (app.api = { variables: { ... }, secrets: [...] } — schema-legal per ApiRefSchema, schema.ts:49-58) loses the whole entry when an explicit composition command re-serializes bos.app.ts via saveBosConfig:

  • bos plugin add / bos plugin remove
  • bos config registry use

ui/host are LowRef-strict (path/development only), so a pathless entry there is meaningless — but api carries authored variables and is meaningful remote-only.

Acceptance

  • A remote-only app.api with variables survives configInputToDescriptor(toConfigInput(descriptor))
  • Integration test alongside the remote-only-auth round-trip case in tests/integration/bos-app-loader.test.ts
  • Decide explicitly whether host/ui get the same treatment (arguably no — nothing authored left when the path is stripped) and pin either way with tests
Lingua principale
TypeScript
Stelle
5
Fork
3
Merge medio
10h 31m
PR unite (30g)
121

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di NEARBuilders/everything-dev

Tutte le issue di NEARBuilders/everything-dev

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.