Skip to content

refactor: remove completed audit bloat - #57

Merged
vnedyalk0v merged 3 commits into
devfrom
refactor/remove-audit-bloat
Jun 23, 2026
Merged

refactor: remove completed audit bloat#57
vnedyalk0v merged 3 commits into
devfrom
refactor/remove-audit-bloat

Conversation

@vnedyalk0v

@vnedyalk0v vnedyalk0v commented Jun 23, 2026

Copy link
Copy Markdown
Owner

Context

This follows the ponytail audit cleanup by removing completed planning artifacts and cutting the remaining unsupported or duplicated workflow surface.

Changes

  • Delete completed plans/ backlog, stale pre-MVP planning docs, and retained /flow-watch spike notes.
  • Replace full duplicated config examples with small project override examples and link docs to the maintained example files.
  • Add short descriptions for each maintained example config so docs remain readable/searchable without duplicating JSON.
  • Add a shared command argument helper and remove duplicated parser blocks from command modules.
  • Remove unsupported config surface: extends, check-level timeoutSeconds, short-sha collision mode, and no-op safety flags.
  • Consolidate repeated template loading, command error formatting, and JSON parse/clone wrappers.
  • Preserve the public renderPrTitle export through src/index.ts.

Verification

  • npm run check passed: JSON, text safety, docs format, typecheck, and 63 Vitest files / 505 tests.
  • Follow-up docs edit: npm run check:docs passed.
  • git diff --check passed.

Self-review

  • Confirmed branch refactor/remove-audit-bloat is the open head branch for PR refactor: remove completed audit bloat #57 against dev.
  • Confirmed unsupported config keys now fail schema validation instead of being accepted as no-ops.
  • Confirmed behavior-preserving template-loader refactor is covered by existing renderer/template tests plus the full suite.

Risk

Medium. Projects using removed compatibility/config fields now fail validation and must switch to the supported fields. Main external-code risk remains deep imports of files deleted earlier in this PR; package public exports remain intact for renderPrTitle.

Rollback

Revert commits da16cb2, 2082f17, and 7e3d4fe.

Linked issues

None.

@vnedyalk0v vnedyalk0v self-assigned this Jun 23, 2026
@vnedyalk0v
vnedyalk0v marked this pull request as ready for review June 23, 2026 07:41
Remove unsupported config surface and no-op safety flags.

Consolidate template loading and simple command error formatting.

Verification: npm run check
Add short descriptions for each maintained example config without restoring duplicated JSON.

Verification: npm run check:docs
@vnedyalk0v

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

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

Reviewed commit: da16cb25c8

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

@vnedyalk0v
vnedyalk0v merged commit 78e8ced into dev Jun 23, 2026
1 check passed
@vnedyalk0v
vnedyalk0v deleted the refactor/remove-audit-bloat branch June 23, 2026 08:26
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