Skip to content

feat(agents): review in slices, check pure refactors, concrete guidelines - #13

Merged
AlexeyShalaev merged 1 commit into
masterfrom
feat/deeper-review
Oct 11, 2026
Merged

AlexeyShalaev merged 1 commit into
masterfrom
feat/deeper-review

Conversation

@AlexeyShalaev

Copy link
Copy Markdown
Member

Takes engineering-assets-template#9 into agents, claude and python-library, byte for byte (python-service is not delivered here).

  • .agents/prompts/review.md reviews a large change in slices, then as a whole, checks every finding at its line, and asks every addition for its reason ("Justify or remove").
  • .agents/prompts/pure-refactor.md (new) checks that a change keeps behaviour; /review-change pure-refactor runs it.
  • AGENTS.md gains working principles; /review-change fans slices out to reviewer subagents; /pr checks the docs before the description.
  • python-library guidelines (python.md, testing.md) gain concrete rules.
  • expected.md, with the libraries' default branches of 2026-10-11: every library is opened with 1 file created (.agents/prompts/pure-refactor.md) and 9 updated. scripts/check_plan.py --status on fresh clones of all 13 matches it; the profiles stay local, and mattermind's and pg-partsmith's ignored files stay ignored.

On merge, distribute opens one sync pull request in each of the 13 libraries. Once they are merged, expected.md goes back to "nothing changes".

scripts/validate.sh --release --public passes.

@github-actions

Copy link
Copy Markdown

touchmark plan

Hub bedrock-python (github.com/1408350934) at fdc326e (PR #13) · touchmark v0.6.0 · strict

Provider Host Read Write Resolve
gh github.com bedrock-python-assets-read[bot] bedrock-python-assets-write[bot]: not checked resolve complete

Scope: every target (13): hub.yml changed

opt_in: assumed: 11 targets without an opt-in file are opted in by targets.yml; an opt-in file with enabled: false opts one out

Outcome Targets
open 13
open (13)
  • gh:bedrock-python/aiokafka-foundation-kit
  • gh:bedrock-python/alembic-gauntlet
  • gh:bedrock-python/clientwright
  • gh:bedrock-python/deadline-budget
  • gh:bedrock-python/grpc-client-kit
  • gh:bedrock-python/grpc-server-kit
  • gh:bedrock-python/idempotency-kit
  • gh:bedrock-python/mattermind
  • gh:bedrock-python/omni-box
  • gh:bedrock-python/pg-partsmith
  • gh:bedrock-python/redis-client-kit
  • gh:bedrock-python/servicewright
  • gh:bedrock-python/sqlalchemy-foundation-kit
This hub pull request changes, across the targets it affects (10)
Change Path Targets
update ⚠ sensitive .claude/agents/reviewer.md 13
update ⚠ sensitive .claude/skills/pr/SKILL.md 13
update ⚠ sensitive .claude/skills/review-change/SKILL.md 13
update .agents/guidelines/python.md 13
update .agents/guidelines/testing.md 13
update .agents/prompts/refactor-check.md 13
update .agents/prompts/review.md 13
update AGENTS.md 13
update CLAUDE.md 13
add .agents/prompts/pure-refactor.md 13

Cost: gh ≈ 39 writes (~1 min) · 1 run

@AlexeyShalaev
AlexeyShalaev merged commit 264f7d1 into master Oct 11, 2026
5 checks passed
@AlexeyShalaev
AlexeyShalaev deleted the feat/deeper-review branch October 11, 2026 19:43
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