sermilion/bill-feature-guard-cleanup
skills/base/bill-feature-guard-cleanup/SKILL.md
Remove feature flags and legacy code after a feature is fully rolled out. Use when a feature flag has been enabled for all users and the legacy path is no longer needed. Safely deletes Legacy classes, removes flag checks, and inlines the winning code path. Use when user mentions remove feature flag, cleanup flag, delete legacy code, or flag fully rolled out.
$ install --global
skillsauthnpx skillsauth add sermilion/mobile-development-plugin bill-feature-guard-cleanupInstall this skill globally with one command. Works with Claude Code, Cursor, and Windsurf.
Security Scan Results
3 of 9 scanners reported clean
Some scanners were skipped, did not run, or reported a non-clean status. Review each row below.
3
Scanners Passed
9
Scanners in report
Clean
TrivyContainer and dependency vulnerability scanner
95%
Clean
SemgrepStatic code analysis for vulnerabilities
95%
Clean
mcp-scan (Snyk)Model Context Protocol security validation
95%
Skipped
Snyk (dep)Open source security scanning
50%
Skipped
Socket.devSupply chain security analysis
50%
Skipped
VirusTotalMulti-engine malware detection
50%
Skipped
CrowdStrikeAdvanced threat intelligence
50%
Skipped
OSV-ScannerOpen Source Vulnerability database check
50%
Skipped
OWASP Dep-Check
50%
Last scanned: Apr 24, 2026, 8:46 PM1.7s1 file scanned
SKILL.md
- name:
- bill-feature-guard-cleanup
- description:
- Remove feature flags and legacy code after a feature is fully rolled out. Use when a feature flag has been enabled for all users and the legacy path is no longer needed. Safely deletes Legacy classes, removes flag checks, and inlines the winning code path. Use when user mentions remove feature flag, cleanup flag, delete legacy code, or flag fully rolled out.
Feature Guard Cleanup
Project Overrides
If .agents/skill-overrides.md exists in the project root and contains a ## bill-feature-guard-cleanup section, read that section and apply it as the highest-priority instruction for this skill. The matching section may refine or replace parts of the default workflow below.
If an AGENTS.md file exists in the project root, apply it as project-wide guidance.
Precedence for this skill: matching .agents/skill-overrides.md section > AGENTS.md > built-in defaults.
When To Use
- Feature flag has been enabled for 100% of users
- No rollback has been needed for an agreed stabilization period
- Product/team has confirmed the feature is permanent
Cleanup Workflow
Step 1: Identify Scope
Before cleanup, gather:
- Feature flag name to remove
- All files referencing this flag (search codebase)
- All
*Legacyclasses/files associated with this flag - Tests that cover the legacy path
Step 2: Verify Safety
Before deleting anything:
- Confirm flag is ON for all users (check flag service/config)
- Confirm no other flags depend on this one
- Confirm no A/B test analysis is still pending
Step 3: Remove (in this order)
- Remove flag checks — replace
if (featureEnabled) { new } else { legacy }with just the new path - Inline the winning path — if a wrapper exists only for the flag check, remove the wrapper
- Delete Legacy files — remove all
*Legacyclasses and their imports - Delete Legacy tests — remove tests that only cover the legacy path
- Remove flag definition — delete the flag from the feature flag registry/enum/config
- Remove unused dependencies — if legacy code pulled in dependencies the new code doesn't need
Step 4: Verify
Run bill-quality-check to ensure nothing is broken.
Patterns
See patterns.md for code examples of each cleanup pattern (conditional, DI/factory, navigation/router).
Checklist
- [ ] Flag is ON for 100% of users
- [ ] Stabilization period has passed
- [ ] All flag references removed from code
- [ ] All Legacy files deleted
- [ ] All Legacy tests deleted
- [ ] Flag definition removed from registry
- [ ]
bill-quality-checkpasses - [ ] No orphaned imports or dependencies
When to Ask User
- Which flag to clean up — if not specified
- Stabilization confirmation — "Has this flag been fully rolled out and stable?"
- Ambiguous ownership — if Legacy code is shared with other flags
Related Skills
sermilion/bill-editorial-assignment-desk
development
VerifiedTrustedCommunity
Use when running a governed editorial assignment desk from Readian recommendations through candidate selection and source-backed story packs.
6SKILL.mdUpdated May 3, 2026
sermilion/bill-unit-test-value-check
testing
VerifiedTrustedCommunity
Use when reviewing unit tests in a file, current changes, or a commit to flag low-value, tautological, or coverage-only tests that do not validate real behavior. Use when user mentions check test quality, review tests, tautological tests, weak tests, or coverage-padding.
6SKILL.mdUpdated Apr 25, 2026
sermilion/bill-skill-remove
data-ai
VerifiedTrustedCommunity
Use when removing an existing skill or platform skill set and cleaning up agent installs, manifests, and supporting links.
6SKILL.mdUpdated Apr 25, 2026
sermilion/bill-quality-check
development
VerifiedTrustedCommunity
Use when you want a generic quality-check entry point that detects the dominant stack in scope and delegates to the matching stack-specific quality-check skill. Use when user mentions run checks, validate, lint, format, quality check, or run quality.
6SKILL.mdUpdated Apr 25, 2026