chore: retire iFlow CLI integration — product discontinued (#3166)#3211
chore: retire iFlow CLI integration — product discontinued (#3166)#3211BenBtg wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
Pull request overview
This PR retires the discontinued iFlow CLI integration by removing its integration subpackage and all related references across the integration registry, catalog, docs, tests, and GitHub issue templates.
Changes:
- Removed the
iflowintegration implementation and unregistered it from the built-in integration registry. - Deleted/updated tests and documentation that referenced
iflow, including the integration catalog entry. - Updated issue templates to stop presenting iFlow CLI as a supported agent option.
Show a summary per file
| File | Description |
|---|---|
tests/test_agent_config_consistency.py |
Removes iflow from agent-config and registrar consistency checks and expected agent key lists. |
tests/integrations/test_registry.py |
Drops iflow from the required integration registry key list. |
tests/integrations/test_integration_iflow.py |
Deletes the integration-specific test coverage for iFlow. |
src/specify_cli/integrations/iflow/__init__.py |
Deletes the iFlow integration implementation. |
src/specify_cli/integrations/__init__.py |
Removes import/registration of the iFlow integration from built-ins. |
integrations/catalog.json |
Removes the iflow entry from the integrations catalog. |
docs/reference/integrations.md |
Removes iFlow CLI from the supported integrations table and multi-install-safe list. |
CHANGELOG.md |
Adds a changelog entry noting the iFlow integration retirement. |
.github/ISSUE_TEMPLATE/feature_request.yml |
Removes iFlow CLI from the “affected agents” dropdown options. |
.github/ISSUE_TEMPLATE/bug_report.yml |
Removes iFlow CLI from the “affected agents” dropdown options. |
.github/ISSUE_TEMPLATE/agent_request.yml |
Removes iFlow CLI from the “currently supported agents” text list. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 11/11 changed files
- Comments generated: 1
- Review effort level: Low
|
|
||
| - chore: retire iFlow CLI integration — product discontinued (#3166) | ||
|
|
||
|
|
mnriem
left a comment
mnriem
left a comment
There was a problem hiding this comment.
Please revert changes to CHANGELOG.md as it is automatically generated
Remove the iFlow CLI integration whose product was shut down: subpackage, registry entry, catalog entry, docs, tests, and issue-template options. Assisted-by: GitHub Copilot (model: Claude Opus 4.8, autonomous) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
BenBtg
force-pushed
the
chore/3166-retire-iflow
branch
from
6b3f274 to
173262c
Compare
|
Thanks — reverted the CHANGELOG.md change (amended the commit, force-pushed). Applied the same revert across the sibling PRs in this batch (#3212, #3213, #3214, #3215, #3216) so none of them touch the auto-generated changelog. Re-requesting review. Posted on behalf of @BenBtg by GitHub Copilot (model: Claude Opus 4.8, autonomous). |
| from .gemini import GeminiIntegration | ||
| from .generic import GenericIntegration | ||
| from .goose import GooseIntegration | ||
| from .hermes import HermesIntegration | ||
| from .iflow import IflowIntegration | ||
| from .junie import JunieIntegration |
mnriem
left a comment
mnriem
left a comment
There was a problem hiding this comment.
Please address Copilot feedback and update PR description to match
# Conflicts: # src/specify_cli/integrations/iflow/__init__.py # tests/integrations/test_integration_iflow.py
|
Addressed the review feedback:
Net diff is removal-only across the integration, catalog, docs, issue templates, and tests — Note: the registry + agent-config consistency suites pass. The 10 failures in Posted on behalf of @BenBtg by GitHub Copilot (model: Claude Opus 4.8, autonomous). |
Closes #3166.
The iFlow CLI product was discontinued (maintenance stopped 2026-03-20, closed 2026-04-17), so this removes the dead integration.
Changes
src/specify_cli/integrations/iflow/subpackage_register()fromintegrations/__init__.pyiflowfromintegrations/catalog.jsoniflowmapping fromextensions/agent-context/agent-context-defaults.jsondocs/reference/integrations.mdtests/integrations/test_integration_iflow.py; dropiflowfromtest_registry.pyandtest_agent_config_consistency.pyCHANGELOG.mdis intentionally untouched (auto-generated).The registry + agent-config consistency suites pass. (The 10 failures in
tests/extensions/test_extension_agent_context.pyare a pre-existing bash syntax error inupdate-agent-context.shonmain, unrelated to this change.)🤖 Opened by GitHub Copilot (model: Claude Opus 4.8) on behalf of @BenBtg.