Skip to content

feat: add agent-native workspace bootstrap - #7

Merged
chasechou007 merged 6 commits into
mainfrom
codex/agent-native-workspace-bootstrap
Aug 3, 2026
Merged

feat: add agent-native workspace bootstrap#7
chasechou007 merged 6 commits into
mainfrom
codex/agent-native-workspace-bootstrap

Conversation

@chasechou007

@chasechou007 chasechou007 commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add package-manager-neutral opendomain init --tools codex workspace bootstrap
  • add managed Codex Skills and bounded AGENTS.md instructions
  • add read-only opendomain doctor and managed-only opendomain update
  • add schema-validated workspace configuration and adapter version compatibility
  • reconcile deselected generated adapters without touching user-owned resources
  • harden managed reads and writes against symlinks, non-regular files, unreadable files, temporary-path attacks, and permission drift
  • record the adoption principle as a proposed Domain Candidate

Validation

  • Node.js 20: 169 tests passed
  • Node.js 22: 169 tests passed
  • opendomain validate: 26 documents passed, one known stale-Candidate warning
  • installed-package smoke passed with healthy Agent integration
  • empty non-Node project bootstrap, doctor, and validation passed without package metadata
  • npm package dry-run: 58 public files (73,703 bytes), no private root OpenSpec, local Codex workspace, or credentials
  • npm audit: 0 vulnerabilities
  • GitHub Actions run 47 passed on Node 20 and 22
  • final Codex Review of bf96d98 found no major issues; all prior review threads are resolved

Boundaries

This PR does not add standalone binaries, Homebrew distribution, other Agent adapters, or automatic Candidate acceptance. The new self-model Candidate remains proposed.

Copy link
Copy Markdown
Contributor 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: 21d9d81458

ℹ️ 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 src/managed-agent-skills.mjs Outdated
Comment thread src/cli.mjs Outdated

Copy link
Copy Markdown
Contributor 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: a14ecdb55d

ℹ️ 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 src/agent-resources.mjs
Comment thread src/managed-agents.mjs Outdated

Copy link
Copy Markdown
Contributor Author

@codex review

Please review the latest commit f615720, including the fixes for deselected generated adapters and non-regular AGENTS.md paths.

@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: f61572042e

ℹ️ 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 src/workspace-config.mjs Outdated
Comment thread src/managed-agent-skills.mjs Outdated

Copy link
Copy Markdown
Contributor Author

@codex review

Please review the latest commit 29c3aa5. CI passes on Node 20 and 22, including installed-package smoke. Focus on the managed read-boundary fixes and their structured-failure regressions.

@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: 29c3aa5601

ℹ️ 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 src/workspace-config.mjs

Copy link
Copy Markdown
Contributor Author

@codex review

Please perform a final review of commit 5ec99a7. CI passes on Node 20 and 22. The latest change removes pre-planner config existence gates and adds dangling-symlink command-matrix coverage.

@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: 5ec99a7d2e

ℹ️ 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 src/atomic-write.mjs Outdated

Copy link
Copy Markdown
Contributor Author

@codex review

Please perform a final review of commit bf96d98. CI passes on Node 20 and 22. The latest change preserves existing POSIX modes across atomic managed-file replacement and covers config, AGENTS, and generated Skills.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Bravo.

Reviewed commit: bf96d98cab

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

@chasechou007
chasechou007 merged commit 675d26d into main Aug 3, 2026
2 checks passed
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