Close four of the gaps the coverage manifest recorded

[giordano-lucas]

#85 and nottelabs/notte-skills#42 have merged; this is now based on main and carries only its own two commits.

Why

The manifest #85 introduced listed five endpoints as "not exposed yet". Four are worth having, so they're commands now and the skip lines are gone — the file working as intended, as a list of decisions to revisit rather than a place gaps go to be forgotten. profiles duplicate stays listed; it's already proposed in #53.

notte personas update --persona-id <id> --name "checkout tester"
notte profiles cookies --profile-id <id>
notte profiles cookies-set --profile-id <id> --file cookies.json
notte usage logs [--endpoint /sessions/start] [--page N]

• personas update is generated: PATCH /personas/{persona_id} is a JSON body behind a $ref, so it needed an endpointMap entry and nothing else.
• profiles cookies-set accepts either shape a cookies file comes in — a bare array (Playwright's storageState, the browser extensions) or an object with a cookies key. Making the caller reshape their own export first would be a papercut for no reason. --source-format and --mode are sent only when passed.
• usage logs exposes --endpoint, --only-current-token and --include-system alongside the shared pagination flags. only_active is deliberately left out: it's the generic listing filter, and a request log is never active or inactive.

--instructions → --run-instructions

The field documents a function for whoever calls it — how long a run takes, what each variable means, which sites it trips over. I had described it as configuration for the self-healing agent, which is what the bare name reads like and is not what it is. Renamed, with the help text and both repos' docs rewritten:

notte functions configure --run-instructions \
  "Takes ~3 min, so call it async. Hits a captcha on the login page every few runs.
   \`query\` is the search term, \`max_items\` caps the results at 50."

The API field is untouched — only the flag is renamed, through a new command-scoped override in the generator so the name stays declared in one place. Free to do: v0.0.37 predates #84, so this flag has never shipped in a release. notte page scrape --instructions is a different flag and is untouched.

Checker fix

The skill check now also requires documentation for a command that both runs and has subcommands. notte usage becomes one in this PR, and the old leaf-only rule would have let it slip out of the check the moment it grew usage logs.

Verification

• personas update renames a real persona on staging and reads back (persona deleted afterwards)
• The other three are covered by mock-server tests asserting the actual request: PATCH method and body, both cookie file shapes, and that unpassed filters aren't sent
• make check-endpoints → ✓ 94 endpoints, 32 recorded exceptions (was 36)
• make check-skills against the updated skills branch → ✓ 90 commands

usage logs could not be verified live. /usage/logs returns 503 Service temporarily unavailable after ~25s to plain curl on staging, before any of this code is involved — worth a look independently of this PR.

🤖 Generated with Claude Code

[greptile-apps]

Greptile Summary

The PR exposes persona updates, profile cookie import/export, and usage-log retrieval while renaming the function metadata flag to --run-instructions.

• Adds generated and hand-written command wiring for the newly exposed API endpoints.
• Accepts wrapped and bare-array cookie exports, with validation preventing accidental cookie clearing.
• Extends endpoint, documentation, generator, and coverage-checker tests.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename	Overview
internal/cmd/profiles.go	Adds profile cookie retrieval and import commands, including support for both documented file shapes and validation against empty cookie sets.
internal/cmd/coveragegaps_test.go	Adds mock-server coverage for persona updates, profile cookie operations, empty cookie validation, and usage-log filters.
internal/cmd/usage.go	Adds the usage logs subcommand with pagination and optional request filters.
internal/cmd/personas.go	Adds and wires the persona update command.
scripts/gen-flags/parser.go	Extends flag generation to support command-scoped flag-name overrides.
scripts/checkcoverage/skills.go	Updates documentation coverage checks to include runnable commands that also have subcommands.

_{Reviews (3): Last reviewed commit: "fix: refuse an empty cookie set instead ..." | Re-trigger Greptile}

[giordano-lucas]

@greptile review