Skip to content

fix(report): assume 15 minutes and $110/hour per review comment - #508

Merged
theCodeDrift merged 1 commit into
mainfrom
fix/report-review-cost-defaults
Oct 15, 2026
Merged

theCodeDrift merged 1 commit into
mainfrom
fix/report-review-cost-defaults

Conversation

@theCodeDrift

Copy link
Copy Markdown
Member

Updates the reviewer-cost defaults taskless report publishes. The worker sets no defaults of its own: the CLI fills in projection.inputs, so changing only the example would leave real reports on the old numbers.

Input Was Now Basis
minutesPerComment 4 15 Conservative estimate: reading the change before a comment is written, plus writing it
reviewerHourlyUsd 120 110 $210k-salary SWE: $210,000 / 2,080 h ≈ $101 base, plus a little overhead

What changes

  • packages/cli/src/report/draft.ts: DEFAULT_PROJECTION_INPUTS.
  • .changeset/report-review-cost-defaults.md: patch release note.

Not covered here

The vendored src/generated/report-v1.schema.json still carries the 4/120 example, because the live tskl.es/api/v1/schema is still serving it. Once the worker deploys, run pnpm --filter @taskless/cli generate:report-schema to bring it in line. Nothing reads the example's values, so this is cosmetic.

Verification

pnpm typecheck, pnpm lint, and the full CLI suite (2297 tests) all pass.

Refs taskless/taskless#305

@theCodeDrift
theCodeDrift merged commit bf27a23 into main Oct 15, 2026
8 checks passed
@theCodeDrift
theCodeDrift deleted the fix/report-review-cost-defaults branch October 15, 2026 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant