Skip to content

Define developer and summary-agent tool profiles - #54

Merged
itsjling merged 2 commits into
mainfrom
codex/issue-033-tool-profiles
Jul 31, 2026
Merged

Define developer and summary-agent tool profiles#54
itsjling merged 2 commits into
mainfrom
codex/issue-033-tool-profiles

Conversation

@itsjling

@itsjling itsjling commented Jul 30, 2026

Copy link
Copy Markdown
Owner

Closes #33

Summary

  • define separate developer and product-summary tool profiles
  • keep summary runs free of broad user config, secrets, network, and tools by default
  • cover allowed, denied, prompted, failed, and timed-out fake MCP calls

Stack

Do not merge this PR until both cross dependencies land.

Verification

  • npm run lint
  • npm test
  • npm run docs:check
  • npm run docs:build
  • npx fallow audit --base codex/issue-031-cursor-sandbox

Stack

  1. Disable unsafe Cursor reviews #51
  2. Define developer and summary-agent tool profiles #54 👈 current
  3. Preserve good notes when agents or files fail #56

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

Copy link
Copy Markdown

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: d85e3c299a

ℹ️ 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".

Comment thread scripts/coding-agents.mjs Outdated
Comment thread scripts/generate-summaries.mjs
@itsjling
itsjling force-pushed the codex/issue-031-cursor-sandbox branch from 5651456 to ce41647 Compare July 30, 2026 17:00
@itsjling
itsjling force-pushed the codex/issue-033-tool-profiles branch from d85e3c2 to a0f36a9 Compare July 30, 2026 17:02
@itsjling

Copy link
Copy Markdown
Owner Author

@codex review

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

Copy link
Copy Markdown

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: a0f36a9f2b

ℹ️ 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".

Comment thread .codex/summary.config.toml.example Outdated
Comment thread .codex/developer.config.toml.example
@itsjling
itsjling force-pushed the codex/issue-031-cursor-sandbox branch from ce41647 to d95bd4e Compare July 30, 2026 17:16
@itsjling
itsjling force-pushed the codex/issue-033-tool-profiles branch from a0f36a9 to 3603071 Compare July 30, 2026 17:17
@itsjling
itsjling changed the base branch from codex/issue-031-cursor-sandbox to main July 31, 2026 03:07
@itsjling itsjling added the automation-reviewed Repo-owned automation changes were reviewed label Jul 31, 2026
@itsjling
itsjling merged commit ac32613 into main Jul 31, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation-reviewed Repo-owned automation changes were reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Define developer and summary-agent tool profiles

1 participant