Skip to content

chore: remove slides tool module and all references - #4

Merged
Brian Krabach (bkrabach) merged 2 commits into
mainfrom
remove-slides-tool
Sep 10, 2025
Merged

chore: remove slides tool module and all references#4
Brian Krabach (bkrabach) merged 2 commits into
mainfrom
remove-slides-tool

Conversation

@bkrabach

Copy link
Copy Markdown
Collaborator

Removed the entire amplifier/slides_tool module as it was identified
for removal. This simplifies the codebase by eliminating ~4,500 lines
of code that are no longer needed.

Changes:

  • Removed amplifier/slides_tool/ directory (17 files)
  • Removed slides-related make targets from Makefile
  • Removed slides-exports optional dependency from pyproject.toml
  • Removed slides tool documentation (docs/AMPLIFIER_SLIDES_TOOL.md)
  • Simplified tests/conftest.py to remove slides-specific fixtures

🤖 Generated with Claude Code

Co-Authored-By: Claude noreply@anthropic.com

Removed the entire amplifier/slides_tool module as it was identified
for removal. This simplifies the codebase by eliminating ~4,500 lines
of code that are no longer needed.
Changes:
- Removed amplifier/slides_tool/ directory (17 files)
- Removed slides-related make targets from Makefile
- Removed slides-exports optional dependency from pyproject.toml
- Removed slides tool documentation (docs/AMPLIFIER_SLIDES_TOOL.md)
- Simplified tests/conftest.py to remove slides-specific fixtures
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
…move-slides-tool
# Conflicts:
#	pyproject.toml
@bkrabach
Brian Krabach (bkrabach) merged commit b4e2417 into mainSep 10, 2025
1 check passed
@bkrabach
Brian Krabach (bkrabach) deleted the remove-slides-tool branch September 10, 2025 22:21
Michael J. Jabbour (michaeljabbour) added a commit to michaeljabbour/amplifier that referenced this pull request Sep 30, 2025
Complete comprehensive technical specification library with all 44 principles:
**People (6 specs)**
- #1 Small AI-first working groups
- #2 Strategic human touchpoints only
- #3 Prompt engineering as core skill
- microsoft#4 Test-based verification over code review
- microsoft#5 Conversation-driven development
- microsoft#6 Human escape hatches always available
**Process (13 specs)**
- microsoft#7 Regenerate, don't edit
- microsoft#8 Contract-first everything
- microsoft#9 Tests as the quality gate
- microsoft#10 Git as safety net
- microsoft#11 Continuous validation with fast feedback
- microsoft#12 Incremental processing as default
- microsoft#13 Parallel exploration by default
- microsoft#14 Context management as discipline
- microsoft#15 Git-based everything
- microsoft#16 Docs define, not describe
- microsoft#17 Prompt versioning and testing
- microsoft#18 Contract evolution with migration paths
- microsoft#19 Cost and token budgeting
**Technology (18 specs)**
- microsoft#20 Self-modifying AI-first codebase
- microsoft#21 Limited and domain-specific by design
- microsoft#22 Layered virtualization
- microsoft#23 Protected self-healing kernel
- microsoft#24 Long-running agent processes
- microsoft#25 Simple interfaces by design
- microsoft#26 Stateless by default
- microsoft#27 Disposable components everywhere
- microsoft#28 CLI-first design
- microsoft#29 Tool ecosystems as extensions
- microsoft#30 Observability baked in
- microsoft#31 Idempotency by design (reference)
- microsoft#32 Error recovery patterns built in
- microsoft#33 Graceful degradation by design
- microsoft#34 Feature flags as deployment strategy
- microsoft#35 Least-privilege automation
- microsoft#36 Dependency pinning and security scanning
- microsoft#37 Declarative over imperative
**Governance (7 specs)**
- microsoft#38 Access control and compliance
- microsoft#39 Metrics and evaluation everywhere
- microsoft#40 Knowledge stewardship and institutional memory
- microsoft#41 Adaptive sandboxing with explicit approvals
- microsoft#42 Data governance and privacy controls
- microsoft#43 Model lifecycle management
- microsoft#44 Self-serve recovery with known-good snapshots
Each specification includes:
- Plain-language definition
- AI-first development rationale
- 4-6 implementation approaches
- 5 good/bad example pairs with working code
- 6 related principles with relationships
- 7 common pitfalls with examples
- Tools organized by category
- 12 actionable checklist items
Statistics:
- 44 specifications totaling ~10,000+ lines
- 220+ good/bad code example pairs
- 240+ implementation approaches
- 300+ documented anti-patterns
- 500+ tools and frameworks
- 250+ cross-principle relationships
Created through parallel AI agent execution demonstrating
Principle microsoft#13 (Parallel Exploration by Default).
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Joi Ito (Joi) pushed a commit to Joi/amplifier that referenced this pull request Oct 31, 2025
* chore: remove slides tool module and all references
Removed the entire amplifier/slides_tool module as it was identified
for removal. This simplifies the codebase by eliminating ~4,500 lines
of code that are no longer needed.
Changes:
- Removed amplifier/slides_tool/ directory (17 files)
- Removed slides-related make targets from Makefile
- Removed slides-exports optional dependency from pyproject.toml
- Removed slides tool documentation (docs/AMPLIFIER_SLIDES_TOOL.md)
- Simplified tests/conftest.py to remove slides-specific fixtures
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
* Merge branch 'main' of https://github.com/microsoft/amplifier into remove-slides-tool
# Conflicts:
#	pyproject.toml
Mollie Munoz (momuno) added a commit to momuno/amplifier that referenced this pull request Nov 20, 2025
… for v0.3.0
Created feature scope and sprint plan for v0.3.0 "Test Case + Basic Regeneration"
focusing on template-based document regeneration with diff preview workflow.
Convergence Session (2025-11-19):
- Defined 5 must-have features for reliable doc regeneration
- Created test case template for amplifier/README.md
- Deferred 21 features to backlog (updated MASTER_BACKLOG to 57 items)
- Captured convergence completion summary
Sprint Planning (v0.3.0):
- Sprint 8 (4 days): Template parser & manual regen command
- Sprint 9 (3 days): Source context & structure preservation
- Sprint 10 (3 days): Change detection & user confirmation
- Integrated issues microsoft#4, microsoft#5, microsoft#6, microsoft#7 into sprint work
Workflow Improvements:
- Fixed convergence-architect to create all 4 required outputs
- Fixed sprint-planner to use proper versioned directory structure
- Added test case template creation to convergence Phase 3
- Added validation checklists to prevent incomplete workflows
Test Case:
- Created templates/amplifier_readme.json as primary test case
- 9 sections covering real-world README structure
- Will validate all v0.3.0 features against actual project
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Mollie Munoz (momuno) added a commit to momuno/amplifier that referenced this pull request Nov 21, 2025
…d polish
Day 4 TDD cycle complete (🔴 RED → 🟢 GREEN → 🔵 REFACTOR):
🔴 RED - Integration Tests (11 tests):
- Full workflow tests (new file, existing file, approval/rejection)
- Auto-approve flag tests
- Output override tests
- Example template validation tests
- Error handling tests (invalid JSON, missing files, permissions)
🟢 GREEN - Implementation Fixes:
- Fixed cli.py to support both Sprint 5 and Sprint 8 template formats
- Added format detection (checks for 'document' vs 'template_version' keys)
- Improved error handling for directory creation permissions
- Enhanced permission error reporting to stderr
🔵 REFACTOR - Polish:
- Improved CLI help text (resolves Issue microsoft#4)
- Enhanced main CLI description with Quick Start guide
- Detailed regen-doc command help with workflow explanation
- All code passes linting with no issues
Test Results:
- 51 Sprint 8 tests passing (15 change_detection + 25 regen_doc + 11 integration)
- 100% of Sprint 8 functionality validated
- Integration tests cover all user workflows end-to-end
Deliverables:
✅ 11 comprehensive integration tests
✅ Support for both template formats (Sprint 5 & Sprint 8)
✅ Clear, helpful CLI help text
✅ Robust error handling and reporting
✅ All tests passing
🤖 Generated with [Amplifier](https://github.com/microsoft/amplifier)
Co-Authored-By: Amplifier <240397093+microsoft-amplifier@users.noreply.github.com>
Sign up for freeto 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

@bkrabach