Skip to content

Remove app/ai folder and inline extracted dependencies#73

Draft
Copilot wants to merge 1 commit intoremove-old-codefrom
copilot/sub-pr-72
Draft

Remove app/ai folder and inline extracted dependencies#73
Copilot wants to merge 1 commit intoremove-old-codefrom
copilot/sub-pr-72

Conversation

Copy link
Contributor

Copilot AI commented Feb 22, 2026

Removes the legacy app/ai pipeline and all associated code, replacing the single cross-module dependency (extract_links) with an inline implementation.

Changes

  • Deleted app/ai/: Entire legacy pipeline folder including pipeline steps, adjudication, claim extraction, retrieval, threads, utils, and all associated tests/docs
  • Deleted app/api/endpoints/research.py and test.py: Endpoints that depended on the old pipeline
  • app/agentic_ai/nodes/format_input.py: Replaced import of extract_links from deleted module with a local regex-based implementation
  • app/api/endpoints/__init__.py / app/main.py: Removed registrations for the now-deleted research and test routers
  • Added app/agentic_ai/tests/nodes/test_extract_links.py: Tests for the new inline extract_links
  • Deleted app/agentic_ai/context/factcheckapi/tests/google_factcheck_gatherer_test.py: Test for removed gatherer
# extract_links moved from app/ai/pipeline/link_context_expander into format_input.py
def extract_links(text: str) -> list[str]:
    url_pattern = r'https?://[^\s<>"{}|\\^`\[\]]+'
    urls = re.findall(url_pattern, text)
    # strips trailing punctuation and deduplicates
    ...

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI mentioned this pull request Feb 22, 2026
Copilot AI changed the title [WIP] Remove old code from the project Remove app/ai folder and inline extracted dependencies Feb 22, 2026
Copilot AI requested a review from caue-paiva February 22, 2026 16:48
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.

2 participants