Skip to content

fix: allow deleting Flow Model despite Session/Agent references - #71

Merged
ShrihariMahabal merged 2 commits into
developfrom
fix/model-delete-link-check
Jul 15, 2026
Merged

fix: allow deleting Flow Model despite Session/Agent references#71
ShrihariMahabal merged 2 commits into
developfrom
fix/model-delete-link-check

Conversation

@ShrihariMahabal

@ShrihariMahabal ShrihariMahabal commented Jul 15, 2026

Copy link
Copy Markdown
Member

Summary

Sessions and Agents keep a historical/required link to a Flow Model, which blocked deleting the model. Adds them to ignore_links_on_delete so the link check no longer blocks it.

@greptile-apps

greptile-apps Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Confidence Score: 5/5

Safe to merge — adding Flow Session to ignore_links_on_delete is a minimal, targeted change that only relaxes the link-check for historical session references.

The one-line change is narrowly scoped: sessions hold read-only historical bookkeeping links to Flow Model, so skipping the link check on delete is correct and carries no operational risk.

No files require special attention.

Reviews (2): Last reviewed commit: "fix: keep Flow Agent link check, only ex..." | Re-trigger Greptile

Comment thread flow/hooks.py Outdated
@ShrihariMahabal
ShrihariMahabal merged commit f86ef4e into develop Jul 15, 2026
4 checks passed
@ShrihariMahabal
ShrihariMahabal deleted the fix/model-delete-link-check branch July 15, 2026 12:40
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.

1 participant