Skip to content

docs(solutions): refresh stale references and prune obsolete doc#712

Merged
tmchow merged 2 commits intomainfrom
worktree-unified-popping-key
Apr 28, 2026
Merged

docs(solutions): refresh stale references and prune obsolete doc#712
tmchow merged 2 commits intomainfrom
worktree-unified-popping-key

Conversation

@tmchow
Copy link
Copy Markdown
Collaborator

@tmchow tmchow commented Apr 28, 2026

Five docs in docs/solutions/ referenced agents, code paths, and converters that no longer exist; one documented a skill that isn't in this repo. Refreshed via ce-compound-refresh.

  • pass-paths-not-content-to-subagents: rename project-standards-reviewer to ce-project-standards-reviewer, ce:review to ce-code-review (post-namespace-migration drift).
  • compound-refresh-skill-improvements: terminology drift mode:autonomous to mode:autofix, auto-archive to auto-delete to match current SKILL.md.
  • colon-namespaced-names-break-windows-paths: trim Layer 2 (src/sync/ no longer exists, consolidated into target writers) and drop Windsurf/OpenClaw from the affected-targets list.
  • cross-platform-model-field-normalization: mark Qwen and OpenClaw as removed (both now use native plugin install per native-plugin-install-strategy-2026-04-19.md).
  • claude-permissions-optimizer-classification-fix: deleted. The skill does not exist in this repo, and git history preserves the fix details if the work is revived elsewhere.

Also expanded the docs/solutions/ bullet in AGENTS.md to name the frontmatter fields (module, tags, problem_type) and note that the directory is relevant when implementing or debugging in documented areas.

24 other docs were verified against current code by parallel investigation subagents and kept without edits.


Compound Engineering
Claude Code

- pass-paths-not-content: rename project-standards-reviewer → ce-project-standards-reviewer, ce:review → ce-code-review
- compound-refresh-skill-improvements: rename autonomous→autofix, archive→delete to match current SKILL.md terminology
- colon-namespaced (Windows paths): drop Layer 2 (src/sync/ removed); update target list to current set
- cross-platform-model-field-normalization: mark Qwen/OpenClaw as removed (now use native plugin install)
- delete claude-permissions-optimizer-classification-fix: skill does not exist in this repo
- AGENTS.md: expand docs/solutions/ bullet with frontmatter fields and search timing for discoverability
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: 7eab38aa53

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

The deleted doc was about a skill not in this repo. Plans cited it for
general principles (pipeline ordering, classification invariants,
negative-signal enumeration) already stated inline in the plan text.
Drop the 8 parenthetical citations and Related-learning entries rather
than restoring a doc tied to a non-existent skill.

Addresses PR #712 review feedback.
@tmchow tmchow merged commit 8cc07ac into main Apr 28, 2026
2 checks passed
michaelvolz pushed a commit to michaelvolz/compound-engineering-plugin-windows-version that referenced this pull request Apr 28, 2026
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