Skip to content

Feature/add agent project support#213

Merged
wcollins merged 6 commits into
itential:mainfrom
nathaniel-itential:feature/add-agent-project-support
Jul 16, 2026
Merged

Feature/add agent project support#213
wcollins merged 6 commits into
itential:mainfrom
nathaniel-itential:feature/add-agent-project-support

Conversation

@nathaniel-itential

Copy link
Copy Markdown
Contributor

Description

Adds agent project support. Needed for asset promotion with the launch of FlowAI.

Type of Change

  • Bug fix
  • New feature
  • Documentation
  • Refactor
  • Chore

Changes Made

Added a resource, service, handler, and runner for agent project support. Added required tests and configured agent projects in ipctl setup as dictated by the contributing doc.

Testing

Ran created service tests and tested against live uat environment.

Checklist

  • Code follows the project's style guidelines
  • Self-review of code has been performed
  • Code has been commented where necessary
  • Tested with make setup or relevant profile
  • Commits follow conventional format (type: subject)
  • No secrets or credentials committed
  • Documentation has been updated accordingly
  • PR has been labeled appropriately (enhancement, bug, documentation, refactor, chore)

@wcollins

Copy link
Copy Markdown
Contributor

@nathaniel-itential @jennlu330

Duplicate of #209 - which was already merged and completed approx. 50% of the changes in this PR. Can you resubmit this PR with the below diff that #209 didn't cover?

  1. create agent-project – with an optional --description flag. This is new at every layer: main has no Create in the service, resource, or runner at all.
  2. delete agent-project – feat: add agent project import/export commands #209 has Delete at the service/resource level, but only uses it internally for import's replace path; it never exposed a delete command to the user.
  3. clear agent-projects – bulk-delete all agent projects. Nothing comparable in feat: add agent project import/export commands #209.
  4. copy agent-project --from --to – cross-server promotion in one step, including re-applying --member permissions on the destination. feat: add agent project import/export commands #209 has no Copy/CopyFrom/CopyTo at all; today you'd have to export from one profile and import to the other manually. Given the PR description says this is "needed for asset promotion with the launch of FlowAI", this is arguably the point of the PR

@nathaniel-itential
nathaniel-itential force-pushed the feature/add-agent-project-support branch from f4e044c to 022f228 Compare July 15, 2026 19:29
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@wcollins

Copy link
Copy Markdown
Contributor

@jennlu330 @nathaniel-itential Everything looks good overall. Build and full test suite pass locally, no conflicts, and the new create/delete/copy/clear commands work as expected. One issue to fix before I merge:

The agent project handler is now registered twice in internal/handlers/handlers.go. PR #209 already added NewAgentProjectHandler(rt, descriptors) under the "Flow Agent handlers" section (around line 145), and this PR adds it again under "Automation Studio handlers" (line 108). The result is that every agent-project command shows up twice in the help output:

Agent Project Commands:
agent-project Create a new agent project.
agent-project Create a new agent project.

This hits all the verbs (get, describe, create, delete, copy, clear, import, export). Execution still works since Cobra dispatches to the first match, but the help text is visibly broken. Since your PR groups agent projects under their own section now, removing the old registration at line 145 is the right fix.

One nit, non-blocking: the new test fixture is under testdata/2023.2/agent-projects/ while the existing agent project fixtures live in agent-project-service/. Might be worth consolidating for consistency.

Fix the duplicate registration and I'll merge.

@nathaniel-itential

Copy link
Copy Markdown
Contributor Author

Will do!

…ate fixtures

Removes the duplicate NewAgentProjectHandler registration under Automation
Studio handlers (already registered under Flow Agent handlers by PR itential#209).
Moves create.success.json fixture into agent-project-service/ alongside
existing fixtures and removes the now-empty agent-projects/ directory.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@wcollins
wcollins merged commit ddcdd39 into itential:main Jul 16, 2026
1 check passed
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