Skip to content

docs: add improve implementation plans - #54

Merged
vnedyalk0v merged 1 commit into
devfrom
docs/improve-plans
Jun 22, 2026
Merged

docs: add improve implementation plans#54
vnedyalk0v merged 1 commit into
devfrom
docs/improve-plans

Conversation

@vnedyalk0v

Copy link
Copy Markdown
Owner

Summary

Adds self-contained implementation plans for the audited improvement findings.

Context

The $improve audit identified four follow-up items: promotion policy drift, raw /flow-check return data, unbounded timeout settings, and stale /flow-watch ADR wording. This PR persists those findings as executor-ready plans.

Changes

  • Added plans index with execution order and rejected findings.
  • Added promotion merge-policy alignment plan.
  • Added /flow-check command return sanitization plan.
  • Added Codeflow timeout-bounds plan.
  • Added /flow-watch ADR alignment plan.

Verification

  • npm run check: passed
  • git diff --check --cached: passed before commit

Self-review

  • Confirmed this is planning/documentation only.
  • Confirmed no runtime code changes were added.
  • Confirmed no dependencies were added.
  • Confirmed no publishing, release automation, deployment, or secrets were added.

Risk

Low. This PR only adds advisor plans under plans/.

Rollback

Revert this PR.

Reviewer notes

Please review whether the plans are scoped correctly, executable by a fresh agent, and ordered appropriately.

Linked issues

None.

@vnedyalk0v vnedyalk0v self-assigned this Jun 22, 2026
@vnedyalk0v
vnedyalk0v marked this pull request as ready for review June 22, 2026 13:49
@vnedyalk0v
vnedyalk0v merged commit 6dd6a40 into dev Jun 22, 2026
1 check passed
@vnedyalk0v
vnedyalk0v deleted the docs/improve-plans branch June 22, 2026 14:21
vnedyalk0v added a commit that referenced this pull request Jun 22, 2026
## Summary
- Align promotion docs with the repository merge settings.
- Sanitize /flow-check command return data so raw stdout/stderr are not
returned.
- Bound Codeflow timeout config in schema and runtime validation.
- Mark the stale /flow-watch ADR default as superseded and mark plans
001-004 DONE.

## Verification
- npm run check
- git diff --check origin/dev..HEAD

## Self-review
- Confirmed the PR is branched from dev and does not target a reserved
branch.
- Confirmed no dependencies, release automation, publishing, or runtime
behavior outside the plan scope were added.
- Confirmed plan 003 uses local constants instead of a new validation
layer.

## Related issues and PRs
- Related issues: none provided.
- Based on improve plans added in PR #54.

## Review comments addressed
- None.

## Decisions made
- Kept plan execution as separate commits and added a final plan-index
status commit.

## Known risks or follow-up work
- Automatic review may still request wording or API-surface adjustments.
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