AgentStack
Browse Sign in
Browse Why AgentStack Sell Docs
Sign in
SKILL verified MIT Self-run

Safe Refactoring

skill-alex7develop-my-agent-skills-safe-refactoring · by Alex7develop

Изменять структуру и внутреннее устройство кода без изменения внешнего поведения, маленькими проверяемыми шагами. Используй при запросах "зарефактори", "упрости этот код", "убери дублирование", "перепиши для читаемости" без изменения функциональности. Не путать с фичами — если поведение должно поменяться, это не рефакторинг, а обычная реализация (см. spec-first / test-driven-development).

↗ Website ↗ Repository
— No reviews yet
0 installs
7 views
0.0% view→install

Install

$ agentstack add skill-alex7develop-my-agent-skills-safe-refactoring

✓ scanned · ✓ verified, works with Claude Code, Cursor, and more.

Security review

✓ Passed

No issues found. Passed automated security review. · v0.1.0 How review works →

  • ✓ Prompt-injection patterns
  • ✓ Secret / credential exfiltration
  • ✓ Dangerous shell & filesystem operations
  • ✓ Untrusted network calls
  • ✓ Known-malicious package signatures

What it can access

  • ✓ Network access No
  • ✓ Filesystem access No
  • ✓ Shell / process execution No
  • ✓ Environment & secrets No
  • ✓ Dynamic code execution No

From automated source analysis of v0.1.0. “Used” means the capability is present in the source — more access means more to trust, not that it’s unsafe.

View the full security report →

Verified badge

Passed review? Show it. Paste this badge into your README, it links to the public security report.

AgentStack Verified badge Links to your public security report.
[![AgentStack Verified](https://agentstack.voostack.com/badges/verified.svg)](https://agentstack.voostack.com/security/report/skill-alex7develop-my-agent-skills-safe-refactoring)

Reliability & compatibility

✓ Security review passed
0 installs to date
— no reviews yet
● 24d ago

Declared compatibility

Claude CodeClaude Desktop

Compatibility is declared by the source manifest. End-to-end runtime verification is coming, see below.

Preview Execution monitoring

We're building live execution health for every listing: tool-call success rate, median latency, uptime, and last-checked timestamps, measured, not self-reported. It isn't live yet, so we don't show numbers we can't stand behind.

How agent discovery & health will work →
Are you the author of Safe Refactoring? Claim this listing to set pricing, connect Stripe payouts, and keep 70% of every sale.
Sign up to claim

About

Safe Refactoring

Ключевое отличие рефакторинга от любой другой правки: внешнее поведение не меняется. Если в процессе хочется "а заодно исправить вот эту логику" — это уже не рефакторинг, и это стоит выделить в отдельный шаг.

Предусловие

Перед началом рефакторинга должны быть тесты, покрывающие текущее поведение. Если тестов нет — сначала написать характеризующие тесты ("characterization tests"), которые фиксируют текущее поведение как есть (даже если оно неидеальное), и только потом начинать менять структуру.

Процесс

  1. Зафиксировать текущее поведение тестами (см. предусловие).
  2. Работать тонкими шагами. Один рефакторинг за раз (переименование,

выделение функции, перемещение кода) — не смешивать несколько видов изменений в одном шаге.

  1. Прогонять тесты после каждого шага. Если тесты не написаны заново —

значит, шаг не должен был менять поведение, и любой красный тест — сигнал немедленно откатиться, а не чинить дальше.

  1. Коммитить после каждого зелёного шага. Маленькие коммиты — это

возможность откатиться на последний работающий шаг, а не переписывать всё с нуля при ошибке.

  1. Отделять структурные изменения от изменений поведения. Если во время

рефакторинга видна реальная фича/фикс — зафиксировать это отдельно и не смешивать с текущим PR.

Частые виды безопасного рефакторинга

  • Выделение функции/метода из длинного блока кода
  • Переименование для ясности (переменные, функции, файлы)
  • Устранение дублирования через общую абстракцию
  • Замена условной логики на полиморфизм/таблицу диспетчеризации
  • Разбиение большого модуля на несколько с чёткими границами

Чек-лист перед тем, как считать рефакторинг завершённым

  • [ ] Все существующие тесты зелёные без изменений в самих тестах
  • [ ] Внешний API/контракт модуля не изменился (если изменился — это не

чистый рефакторинг, нужно явно об этом сказать)

  • [ ] Каждый шаг был закоммичен отдельно, коммиты можно откатить по одному
  • [ ] Нет смешанных изменений "рефакторинг + фича" в одном шаге

Source & license

This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.

Install and usage instructions live in the source repository linked above.

Reviews

No reviews yet, be the first.

Write a review

Versions

  • v0.1.0 Imported from the upstream source.

More in Agent Skills