Skip to content

Document verbosity levels 2 and 4 (per-field LLM duration) - #676

Open
fscelliott wants to merge 4 commits into
v0from
doc/pr-3412-verbosity-duration
Open

Document verbosity levels 2 and 4 (per-field LLM duration)#676
fscelliott wants to merge 4 commits into
v0from
doc/pr-3412-verbosity-duration

Conversation

@fscelliott

Copy link
Copy Markdown
Collaborator

Summary

  • Adds rows for verbosity levels 2 and 4 to the Parameters table in verbosity.md
  • Level 2: level 1 + per-field duration (ms) for LLM-extracted fields
  • Level 4: level 3 + per-field duration
  • Includes a TODO comment to (1) add example output for query-group/list fields and (2) investigate level 5 seen in PR 3412 tests

Refs sensible-hq/sensible#3412

Test plan

  • Verify rendered table looks correct in ReadMe preview
  • Confirm duration field name and unit (ms) with eng
  • Follow up on level 5 investigation before removing TODO

🤖 Generated with Claude Code

fscelliott and others added 4 commits August 13, 2026 13:48
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Fix property name: `duration` → `durationMs` in levels 2 and 4 descriptions
- Add verbosity level 2 example showing durationMs on query-group and list fields
- Add section headings to the examples section
- Fix two pre-existing vale errors in level 3 description (quote punctuation, unit spacing)
- Add session checklist with remaining test tasks

Based on sensible-hq/sensible#3412.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@fscelliott
fscelliott force-pushed the doc/pr-3412-verbosity-duration branch from ffc73bd to 75ccd7a Compare August 13, 2026 21:25
@fscelliott

Copy link
Copy Markdown
Collaborator Author

CANCEL FOR NOW: not documenting b/c we want to reimplement more nicely and with finer grained time reporting https://github.com/sensible-hq/sensible/pull/3412#issuecomment-5285725605

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant