Skip to content

Document v0.1 scope and integration stubs#91

Open
jmcte wants to merge 1 commit into
mainfrom
codex/issue-85-scope-docs
Open

Document v0.1 scope and integration stubs#91
jmcte wants to merge 1 commit into
mainfrom
codex/issue-85-scope-docs

Conversation

@jmcte
Copy link
Copy Markdown
Contributor

@jmcte jmcte commented May 23, 2026

Summary

  • document the fixture-mode v0.1 scope and explicitly call out live adapter/live export stubs
  • add live MailPlus adapter and live export integration guides for Phase 2 contributors
  • add Keep-a-Changelog release notes and versioning policy
  • expose mailplus_intelligence.__version__ from package metadata
  • link release-v0.1 / Phase 2 tracking from README and AGENTS

Verification

  • PYTHONPATH=src python3.12 -m unittest discover -s tests -p 'test_runtime.py' -v
  • bash scripts/ci/run-fast-checks.sh

Closes #85.

@jmcte jmcte requested a review from pheidon as a code owner May 23, 2026 21:26
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 834af0ae43

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

return SyncBatch(
source_name=f"mailplus:{config.user}",
cursor=cursor,
next_cursor="123",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Update SyncBatch example to avoid invalid next_cursor arg

The worked IMAP stub currently constructs SyncBatch with next_cursor, but SyncBatch only defines source_name, cursor, and messages in src/mailplus_intelligence/sync.py (SyncBatch dataclass). Anyone implementing Phase 2 by following this guide will hit TypeError: __init__() got an unexpected keyword argument 'next_cursor' at runtime, so the integration instructions are not executable as written.

Useful? React with 👍 / 👎.

@athena-omt athena-omt added area:ai AI/model/agent behavior surface. lane:daedalus Daedalus implementation/forge lane. review:athena Athena review governance requested. risk:medium Medium-risk change; normal care required. state:waiting-checks Waiting for CI/check status to settle. status:needs-review PR is ready for Athena review. labels May 23, 2026
@pheidon pheidon added state:needs-repair PR needs repair before review can proceed. codex:P2 Codex Connector P2 finding. priority:P2 Codex Connector P2; blocks execution until Athena validates. and removed state:waiting-checks Waiting for CI/check status to settle. labels May 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:ai AI/model/agent behavior surface. codex:P2 Codex Connector P2 finding. lane:daedalus Daedalus implementation/forge lane. priority:P2 Codex Connector P2; blocks execution until Athena validates. review:athena Athena review governance requested. risk:medium Medium-risk change; normal care required. state:needs-repair PR needs repair before review can proceed. status:needs-review PR is ready for Athena review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

R4: Honest scope docs, CHANGELOG, and integration guides for stubbed surfaces

3 participants