Skip to content

feat(cli): Add feast projects delete command (closes #5095)#6318

Open
mailtoboggavarapu-coder wants to merge 4 commits intofeast-dev:masterfrom
mailtoboggavarapu-coder:feat/expose-project-delete-cli-v2
Open

feat(cli): Add feast projects delete command (closes #5095)#6318
mailtoboggavarapu-coder wants to merge 4 commits intofeast-dev:masterfrom
mailtoboggavarapu-coder:feat/expose-project-delete-cli-v2

Conversation

@mailtoboggavarapu-coder
Copy link
Copy Markdown

@mailtoboggavarapu-coder mailtoboggavarapu-coder commented Apr 22, 2026

Summary

Closes #5095

Exposes project deletion through the CLI by adding a new feast projects delete <name> subcommand to the existing projects group.

What's changed

Added project_delete command to sdk/python/feast/cli/projects.py:

Usage: feast projects delete [OPTIONS] NAME

  Delete a project and all its resources from the registry.

Options:
  -y, --yes  Skip confirmation prompt and delete immediately.
  --help     Show this message and exit.

Design notes

  • Uses store.registry.delete_project(name, commit=True) via the lazy-init registry property (the abstract method exists in BaseRegistry and is implemented in all concrete registries)
  • Adds an interactive confirmation prompt with click.confirm — can be skipped with --yes / -y for scripting
  • Catches both FeastObjectNotFoundException and ProjectNotFoundException so the CLI exits cleanly (exit code 1) for either variant raised by concrete registry implementations
  • Follows existing CLI patterns in projects.py

Example

# Interactive (prompts for confirmation)
feast projects delete my-project

# Non-interactive
feast projects delete my-project --yes

Checklist

  • Added delete command to projects_cmd group
  • Confirmation prompt with --yes bypass flag
  • Error handling for non-existent projects
  • Follows existing CLI patterns in projects.py

Open in Devin Review

Exposes project deletion via the CLI. The new `feast projects delete <name>`
command calls store.registry.delete_project() which is already implemented
in all concrete registries.

- Adds interactive confirmation prompt with --yes/-y flag for scripting
- Catches both FeastObjectNotFoundException and ProjectNotFoundException
- Exits with code 1 for non-existent projects

Signed-off-by: Venkateswarlu Boggavarapu <mailtoboggavarapu@gmail.com>
devin-ai-integration[bot]

This comment was marked as resolved.

…tation

Addresses Devin Review findings:
- Fix project_list: use utils.tags_list_to_dict() instead of manual tag.split(), and print tabulated output matching master
- Fix project_current: call store.get_project(name=None) with error handling instead of store.project

Both functions now match master branch exactly. Only project_delete is new code from this PR.

Signed-off-by: Venkateswarlu Boggavarapu <mailtoboggavarapu@gmail.com>
@mailtoboggavarapu-coder
Copy link
Copy Markdown
Author

Thanks for the detailed review, @devin-ai-integration! Fixed both issues in commit 21f7f6a:

  1. project_listutils.py_object_to_proto + broken tag parsing: Updated to match master exactly: uses utils.tags_list_to_dict(tags) for tag filtering and prints tabulated output via tabulate.
  2. project_current — simplified incorrectly: Fixed to match master (calls store.get_project(name=None) with try/except and YAML-formatted output).

The only new code is the project_delete command itself.

@mailtoboggavarapu-coder
Copy link
Copy Markdown
Author

Hi team! Friendly ping on this PR — all 18 CI checks are passing and there are no conflicts with the base branch.

This adds a feast projects delete command (closes #5095) with an interactive confirmation prompt and a --yes / -y flag for scripting. Happy to iterate on any feedback!

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.

Expose project deletion to CLI

1 participant