Skip to content

Rename CARE agents to descriptive names and add description fields to match agents on Alpha release#58

Open
sanzog03 wants to merge 2 commits intodevelopfrom
enhance/rename-agents
Open

Rename CARE agents to descriptive names and add description fields to match agents on Alpha release#58
sanzog03 wants to merge 2 commits intodevelopfrom
enhance/rename-agents

Conversation

@sanzog03
Copy link
Copy Markdown
Collaborator

@sanzog03 sanzog03 commented Apr 1, 2026

Summary

Renames all CARE agents from internal naming conventions (e.g., CMRCareAgent, AstroSearchAgent, PDSSearchAgent) to more descriptive, user-facing names (e.g., CMRDataSearchAgent, AstroDataSearchAgent, PlanetaryDataSearchAgent) reflecting the ones in akd-alpha. Also adds CodeSearchAgent to the public API and includes new tests for it.

What it does

  • Renames agents to clearly communicate their purpose:
    • AstroSearchAgentAstroDataSearchAgent
    • CMRCareAgentCMRDataSearchAgent
    • PDSSearchAgentPlanetaryDataSearchAgent
    • CodeSearchAgent (newly exported)
  • Replaces print statements with loguru logger in agent __main__ blocks.
  • Updates all references in docs (CONTRIBUTING.md, HUMAN-IN-THE-LOOP.md) and tests to use the new names.

How it does this

  • Renames classes, input/output schemas, and config classes across all four agent modules (astro_search_care.py, cmr_care.py, pds_search_care.py, code_search_care.py).
  • Updates akd_ext/agents/init.py to export the renamed classes and adds CodeSearchAgent to the public API (__all__).
  • Updates documentation files to reflect the new class names.
  • Updates all corresponding test files to import and use the renamed classes.

Testing

  • Updated all existing agent tests to use the renamed classes.
  • Added new test file tests/agents/test_code_search_care.py with parametrized test cases for CodeSearchAgent.
  • All 30 tests pass with uv run pytest ./tests/agents/.

@sanzog03 sanzog03 requested a review from NISH1001 April 1, 2026 19:57
Copy link
Copy Markdown
Collaborator

@NISH1001 NISH1001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sanzog03 thanks for this. Along this line: can you also test the "config.name" field to see if it reflects on what we wnat the name to be when frontend displays?

Copy link
Copy Markdown
Collaborator

@NISH1001 NISH1001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also: akd_ext.agents.data_search namespace should be there.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

akd_ext.agents.data_search.astro.py

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

akd_ext.agents.data_search.cmr

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

akd_ext.agents.data_search.pds

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

akd_ext.agents.code_search

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