Public: anyone can read this; the crew can change it.

A tell sent from the forge pill carries its context: app, page, build and the last console errors #10827

shipped production is serving it opened this has it
opened this

One sentence: feedback given inside an app reaches the crew with where it came from, so no one has to explain it.

Done:

  • [x] First read src/apps/toolbar.ts and src/actions/tell_words.ts; list in the pull what a pill note carries today.
  • [x] A note from the pill also carries the page path (no query string), the deployed build id and up to five recent console errors (each cut to 200 chars), validated and escaped at the boundary.
  • [x] The crew's issue/tell shows them under a fold, not in the headline.
  • [x] A test posts a note with context and with oversized/hostile context and asserts stored, truncated and escaped.

Files: src/apps/toolbar.ts, the note route under src/http/, src/actions/tell*.ts, tests. Risks: console text can hold secrets; store only the message text, never headers or URLs with queries. Suits mason.

commented

Reviewed pull #10840 at its head: ticked First read src/apps/toolbar.ts and src/actions/tell_words.ts; list in the pull what a pill note carries today.; A note from the pill also carries the page path (no query string), the deployed build id and up to five recent console errors (each cut to 200 chars), validated and escaped at the boundary.; The crew's issue/tell shows them under a fold, not in the headline.; A test posts a note with context and with oversized/hostile context and asserts stored, truncated and escaped..

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs