Skip to content

docs: fix config drift in on-call module#25

Merged
ysyneu merged 1 commit intomainfrom
doc-review/2026-03-23-120000
Mar 23, 2026
Merged

docs: fix config drift in on-call module#25
ysyneu merged 1 commit intomainfrom
doc-review/2026-03-23-120000

Conversation

@ysyneu
Copy link
Copy Markdown
Contributor

@ysyneu ysyneu commented Mar 23, 2026

Doc Review — 2026-03-23

Mode: audit (scope: on-call)
Findings: 9 total — 1 high, 4 medium, 4 low
Applied: 4 (1 high, 3 medium)

Changes

Finding Type Doc Action
f001 config_drift custom-fields.mdx Text field limit 500 → 3000 chars
f002 config_drift custom-actions.mdx Custom actions per channel 3 → 5
f003 config_drift search-view-incident.mdx Query window: "1 month" → tier-based (Free=30d, Standard=180d, Pro=366d)
f005 config_drift insights.mdx Export limit 100,000 → 10,000

Skipped

Finding Reason
f004 User confirmed 4MB image upload limit is correct (frontend has stale 10MB check)
f006 Inhibition 10-min scope claim could not be verified in code — needs manual review
f007 Fair rotation feature — needs frontend visibility confirmation
f008 Hourly/monthly rotation units — backend-only, frontend uses raw seconds
f009 Mobile Critical Alerts (iOS) — low severity, needs deeper mobile app investigation

Review checklist

  • Verify config field names and defaults match current code
  • Check that feature descriptions accurately reflect user-facing behavior
  • Ensure tone and style match the rest of the docs

Mapping warnings

None

🤖 Generated with Claude Code

Correct 4 verified documentation inaccuracies found via doc-review audit:
- Custom field text limit: 500 → 3000 chars (matches backend validation)
- Custom actions per channel: 3 → 5 (matches frontend limit)
- Query time window: fixed "1 month" → tier-based (30d/180d/366d)
- Export limit: 100,000 → 10,000 (matches frontend enforcement)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ysyneu ysyneu merged commit cb6cc16 into main Mar 23, 2026
@ysyneu ysyneu deleted the doc-review/2026-03-23-120000 branch March 23, 2026 06:15
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