Skip to content

docs: Document Todoist OAuth auth setup#16

Merged
amix merged 1 commit into
amix/comms-todoist-oauthfrom
amix/comms-oauth-docs
Jun 5, 2026
Merged

docs: Document Todoist OAuth auth setup#16
amix merged 1 commit into
amix/comms-todoist-oauthfrom
amix/comms-oauth-docs

Conversation

@amix

@amix amix commented Jun 4, 2026

Copy link
Copy Markdown
Member

Context

Follow-up to the Todoist OAuth implementation PR.

What was changed

  • Clarifies the default Comms OAuth grant
  • Documents supported non-production OAuth hosts
  • Notes that custom Comms hosts should use COMMS_API_TOKEN

Testing

  • npm run lint:check
  • doistbot review staged (one local pass, copy issue addressed)

@amix amix requested a review from doistbot June 4, 2026 18:54
@doistbot doistbot requested a review from engfragui June 4, 2026 18:54

@doistbot doistbot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Amir for documenting the Todoist OAuth setup and fallback behaviors 😊.

Few things worth tightening:

  • Clarify that custom hosts also require COMMS_BASE_URL alongside COMMS_API_TOKEN so requests aren't mistakenly routed to production.
  • Sync the new auth documentation to src/lib/skills/content.ts and regenerate SKILL.md so the agent-facing docs match the README.

Share FeedbackReview Logs

Comment thread README.md
Comment thread README.md
@amix amix force-pushed the amix/comms-oauth-docs branch 2 times, most recently from 4cf1e04 to d1267ad Compare June 4, 2026 19:05
@amix

amix commented Jun 4, 2026

Copy link
Copy Markdown
Member Author

Posted by Claude Code/Codex on behalf of Amir:

Addressed the Doistbot notes in this stacked docs PR:

  • The README now calls out that custom Comms hosts need COMMS_BASE_URL alongside COMMS_API_TOKEN.
  • The agent-facing skill docs were synced via src/lib/skills/content.ts and regenerated skills/comms-cli/SKILL.md.

Not requesting another Doistbot round per Amir's direction.

Clarify the default Comms OAuth grant and the supported non-production OAuth hosts.
@amix amix force-pushed the amix/comms-oauth-docs branch from d1267ad to babf07d Compare June 4, 2026 19:58
@amix amix merged commit 728b7a1 into amix/comms-todoist-oauth Jun 5, 2026
5 checks passed
@amix amix deleted the amix/comms-oauth-docs branch June 5, 2026 09:00
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.

3 participants