chore: v2.0.0 cleanup — drop dead .codex-plugin and consolidate changelog#89
Merged
Merged
Conversation
Follow-up to #27 / #50 — v2.0.0 cleanup audit found two leftovers. - Delete `.codex-plugin/plugin.json`. PR #51 (closing #27) explicitly punted on this manifest because "the issue scope is explicit and `.codex-plugin/` is not in the deletion list" — but its sibling artifacts (the Claude marketplace manifest and the `codex-skill/` mirror) were deleted, leaving this file with a broken `./codex-skill` pointer to a directory that no longer exists. Removing it makes the v2.0.0 changelog claim "Removed v1 distribution artifacts (#27)" fully accurate. - Consolidate the changelog. Migrate the v0.1–v1.3 history from `docs/changelog.md` into the root `CHANGELOG.md` (which previously held only the v2.0.0 entry), delete the docs copy, and drop the Changelog entry from `mkdocs.yml` nav. Single source of truth at the repo root, no duplication risk for future releases. `ruff check src/ tests/` clean. `pytest -q` 391 passed, 1 skipped. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Two leftovers found in the post-v2.0.0 audit:
Delete
.codex-plugin/plugin.json. PR chore: delete v1 distribution artifacts #51 (closing Delete v1 distribution artifacts #27 — Delete v1 distribution artifacts) explicitly punted on this manifest, quoting from the PR body: ".codex-plugin/plugin.json still has 'skills': './codex-skill' pointing at a now-deleted directory. Left as-is — the issue scope is explicit and .codex-plugin/ is not in the deletion list." But its sibling artifacts (.claude-plugin/marketplace.json,plugins/wealthbox-crm/,codex-skill/) were all deleted, leaving this file dangling with a broken pointer. Removing it makes the v2.0.0 changelog claim "Removed v1 distribution artifacts (Delete v1 distribution artifacts #27)" fully accurate.Consolidate the changelog. Two changelogs co-existed after PR release: prepare v2.0.0 #88:
CHANGELOG.mdat the repo root (just v2.0.0) anddocs/changelog.md(v0.1–v1.3 history; wired into the mkdocs nav). Merged the v1 history into rootCHANGELOG.md, deleteddocs/changelog.md, and dropped itsmkdocs.ymlnav entry. Single source of truth at the repo root, no duplication risk for future releases.Test plan
ruff check src/ tests/→ All checks passedpytest -q→ 391 passed, 1 skipped.codex-pluginoutside the historical v1.2.0 entry inCHANGELOG.md🤖 Generated with Claude Code