Skip to content

[Docs] Explain partial Slack history and recovery - #3677

Draft
roomote-community[bot] wants to merge 1 commit into
developfrom
fix/docs-slack-partial-history-20261106
Draft

roomote-community[bot] wants to merge 1 commit into
developfrom
fix/docs-slack-partial-history-20261106

Conversation

@roomote-community

Copy link
Copy Markdown
Contributor

​Opened on behalf of @mrubens. Follow up by mentioning @roomote, in the web UI, or in Discord.

Related issue

Daily documentation accuracy audit for 2026-11-05T03:00:36.542Z through 2026-11-06T03:00:36.542Z; no linked issue.

Why this PR exists

  • A maintainer explicitly invited this PR in the linked issue or discussion
  • I am a maintainer / this is internal Roomote work

What changed

Explain partial Slack channel history, including potentially missing older messages and thread replies, independent continuation reads, deduplication, and recovery through narrower time ranges or specific thread links. This helps readers interpret incomplete summaries without assuming the requested range was fully read. No runtime behavior changes.

The audit reviewed all four PRs merged in the exact window: #3645 (ad72e01b), #3674 (345f05ee), #3675 (7d56d859), and production promotion #3676 (7dc2847b). Behavior was traced through the Slack notifier, API lookup and response-size budget, and session/task tool contracts. The promotion also first shipped earlier #3646 and #3666; Slack discovery guidance is already included in the release docs, and the Copilot endpoint repair changes no user configuration. Existing GitHub docs correctly distinguish read-only task MCP from the authorized source-control write workflow.

Compared fetched develop 7d56d859 with shipped main/v1.20.1 7dc2847b; their trees differ only in the changelog. Open documentation PRs were checked for overlap. In particular, #3669's discovery correction is already present in the release and does not cover partial history.

How it was tested

  • mise exec -- pnpm --filter @roomote/docs check passed: pinned Mintlify build validation and no broken links.
  • mise exec -- pnpm --filter @roomote/docs check-cookbook-index passed: seven recipes current.
  • git diff --check passed.
  • Opened the changed Slack guide in the actual local Mintlify browser preview and verified the new section rendered. Validation initially invalidated generated preview assets; restarting the existing preview and reloading the browser restored the page. No screenshots or simulated proof are claimed; semantic accuracy is established from implementation tracing.
  • Normal pre-push checks passed: oxlint, web/worker residual ESLint, all 27 fast typecheck tasks, and Knip.
  • The pre-commit judgement hook allowed the commit but left two prose criteria incomplete because provider/evidence evaluation was unavailable. This is not a full judgement pass.
  • Live Slack rate-limit recovery, Copilot inference, full Vitest, and formatting-inclusive pnpm lint / pnpm check-types were not run for this docs-only correction. Repository source and release history were audited; live production deployment state was not independently verified.

Checklist

  • The PR title follows the repo convention: [Fix], [Feat], [Improve], [Refactor], [Docs], or [Chore] followed by a user-facing description
  • This PR is small and scoped to one change
  • pnpm lint and pnpm check-types pass locally
    • Normal fast pre-push gates and docs-specific checks passed; full formatting-inclusive commands were not run.
  • I added tests or included a clear manual validation note above
  • I removed secrets, tokens, private keys, and customer data from code, logs, and screenshots
  • If this change should appear in the changelog, I ran pnpm changeset
    • Docs-only correction; no changeset needed.

@roomote-community

roomote-community Bot commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor Author

No issues found. See session

Reviewed the Slack guide addition against the bounded history reader, independent history/response-size continuations, and thread-reply recovery notes. The documentation matches the implementation. Current-head docs validation, lint, type checks, and tests passed in CI.

Reviewed 8b83581

This branch has not been deployed

No deployments
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