Skip to content

chore: remove devcontainer and auto-delegation tooling - #640

Merged
jamiesun merged 1 commit into
mainfrom
chore/remove-devcontainer-and-auto-delegation
Oct 8, 2026
Merged

jamiesun merged 1 commit into
mainfrom
chore/remove-devcontainer-and-auto-delegation

Conversation

@jamiesun

@jamiesun jamiesun commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove .devcontainer configuration (devcontainer.json, docker-compose.yml).
  • Remove auto-delegation skills (orchestrate-roadmap, review-pr, groom-roadmap).
  • Remove scripts/agent-roadmap-quality-metrics.sh.
  • Modernize .github/review-checklists/agent-roadmap.md to standard .github/review-checklists/pull-request.md.
  • Update .agents/README.md, AGENT.md, .github/copilot-instructions.md, docs-site/, and docs/roadmap*.md to remove references to the deleted skills and dev container, and document running integration tests via make test-integration-pg.

Verification

  • go test -short ./... passed.
  • golangci-lint run passed with 0 issues.
  • mdbook build docs-site and lychee --offline passed with 0 errors.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-08T15:43:13.571835Z 9446bc5 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@github-actions github-actions Bot added the doc label Oct 8, 2026
@jamiesun
jamiesun enabled auto-merge October 8, 2026 15:41
@jamiesun
jamiesun added this pull request to the merge queue Oct 8, 2026

@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: 9446bc5b43

ℹ️ 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 docs/roadmap.zh.md
- 调度优先级:先 P1(`M2 → M3 → M8 → M9`,均已交付),再 P2(已置顶交付 `M13` 文档站点与 `M4` agent 质量门禁;`M7` 已交付(上游评估收口、协议路径 CI 验收闭环);当前优先收尾 `M14`,其后 `M10`(`M5` 计划批次 M5.1–M5.4 已交付,余量按需驱动)),最后 P3(`M6 / M11 / M12`);同优先级里程碑按依赖与可执行性取,P2/P3 仅在更高优先级里程碑无可执行子任务时填充。**M13 置顶依据**:用户指令将 mdbook 文档站点列为优先实现,先收编散落文档并对外呈现既有能力(含 EAP 套件)。EAP 套件优先续接 M1(EAP-TLS):先 PEAP-MSCHAPv2(兼容)、再 EAP-TTLS(后端适配),TLS 1.3 / TEAP / EAP-PWD 列为中长期 / 按需。
- 单次 agent 任务只认领一个未勾选子任务(最小闭环),完成后在本文件勾选并在 PR 引用里程碑编号。
- **自我迭代**:每轮交付后由 [`.agents/skills/groom-roadmap/SKILL.md`](../.agents/skills/groom-roadmap/SKILL.md) 勾选已交付项、更新里程碑状态、回填/拆分/重排子任务,并保持本文件与功能清单状态一致。
- **路线图维护**:每轮交付后勾选已交付项、更新里程碑状态、回填/拆分/重排子任务,并保持本文件与功能清单状态一致。

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 Retire deleted tooling from the roadmap

Keep the roadmap consistent with this removal: a repo-wide search at this commit shows that M4.8 still claims scripts/agent-roadmap-quality-metrics.sh is delivered, while M4.10/M4.13 still describe the deleted orchestration/review/groom skills and old checklist as active deliverables. This leaves completed milestones pointing contributors to capabilities and files that no longer exist; mark those entries as retired or rewrite them to describe the remaining process.

Useful? React with 👍 / 👎.

Merged via the queue into main with commit b537e98 Oct 8, 2026
14 checks passed
@jamiesun
jamiesun deleted the chore/remove-devcontainer-and-auto-delegation branch October 8, 2026 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant