Skip to content

Conversation

@S0okJu
Copy link
Collaborator

@S0okJu S0okJu commented Sep 4, 2025

Overview

  • Add delete project tool.

Key Changes

  • Add delete_project tool
  • Tests are upaded and passing.

Related Issues

Additional context

  • Even if asked with the project name, the LLM calls get_tool first and then uses that data to delete the project.
image

@S0okJu S0okJu added the feature Request for new feature or functionality enhancement label Sep 4, 2025
@S0okJu S0okJu changed the title Feat/identity project feat: Add delete project tool Sep 4, 2025
@platanus-kr
Copy link
Collaborator

여담이긴 한데 프로젝트를 삭제 할 때 프로젝트'만' 삭제하면 각 컴포넌트에서 orphan 리소스가 발생합니다.
대부분의 리소스들이 프로젝트에 종속되어 있는데 이 경우 '점유는 하고있지만 조회는 되지 않는 리소스'가 발생할 수도 있어요.
후에 이 기능에 프로젝트와 연결된 리소스들을 조회하거나 삭제 할 수 있는 로직 또는 프롬프트를 추가하면 좋을것 같습니다.

LGTM🔥

@S0okJu
Copy link
Collaborator Author

S0okJu commented Sep 4, 2025

@platanus-kr

고아 리소스도 고려한다면 다른 서비스의 도구를 활용하는 것도 생각해야겠네요.
추가적으로 이슈 열어서 다음에 수정하겠습니다.

@S0okJu S0okJu merged commit 5f17b3f into develop Sep 5, 2025
6 checks passed
halucinor pushed a commit that referenced this pull request Oct 23, 2025
* feat(identity): Add get-project, get-projects tool(#58)

- Add get-project, get-projects tool
- Test code are updated and passing

* chore(identity): ruff format(#58)

* chore(identity): ruff format(#58)

* fix(identity): Remove _version.py(#58)

* feat(identity): Add create project tool(#58)

- Add create-project tool
- Tests are updated ans passing

* fix(identity): Fix unnecessary test codes(#?58)

* feat(identity): Add delete project tool(#58)
halucinor pushed a commit that referenced this pull request Oct 24, 2025
* feat(identity): Add get-project, get-projects tool(#58)

- Add get-project, get-projects tool
- Test code are updated and passing

* chore(identity): ruff format(#58)

* chore(identity): ruff format(#58)

* fix(identity): Remove _version.py(#58)

* feat(identity): Add create project tool(#58)

- Add create-project tool
- Tests are updated ans passing

* fix(identity): Fix unnecessary test codes(#?58)

* feat(identity): Add delete project tool(#58)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Request for new feature or functionality enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants