fix(hook-decorator): upgrade django-lifecycle and remove type: ignore[misc] comments

[afsuyadi]

Thanks for submitting a PR! Please check the boxes below:

• [✔] I have read the Contributing Guide.
• [ ] I have added information to docs/ if required so people know about the feature.
• [✔] I have filled in the "Changes" section below.
• [✔] I have filled in the "How did you test this code" section below.

Changes

Contributes to #5240

1. Upgraded django-lifecycle from 1.2.4 to 1.2.5 in pyproject.toml (both the version constraint and the pinned dev dependency)
2. Removed 24 # type: ignore[misc] comments from @hook-decorated methods across 10 files

How did you test this code?

1. Run make typecheck; pass with no [misc] errors
2. Ran make test locally for those 10 files; 2041 tests passed, 0 failures.

Cli:
DJANGO_SETTINGS_MODULE=app.settings.test ./flagsmith/api/.venv/bin/pytest tests/unit/audit/ tests/unit/environments/ tests/unit/features/ tests/unit/projects/ tests/unit/users/ tests/unit/integrations/

[vercel]

@afsuyadi is attempting to deploy a commit to the Flagsmith Team on Vercel.

A member of the Team first needs to authorize it.

[emyller]

Hi @afsuyadi thanks for this contribution! I've unblocked uv lock (7ee256e) to solve this error, but we've now uncovered a few typing errors. 😬

How do you think we should proceed?

[coderabbitai]

📝 Walkthrough

Walkthrough

This pull request bumps the pinned django-lifecycle dependency version from 1.2.4 to 1.2.5 in api/pyproject.toml, and removes now-unneeded # type: ignore[misc] comments from @hook(...) decorators across numerous Django model files (audit, environments, experimentation, features, feature external resources, release pipelines, workflows, github integrations, projects, and users). One hook call's keyword-argument formatting was also reformatted. No runtime logic or public API signatures were changed.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Changes

Cohort / File(s)	Change Summary
Dependency bump api/pyproject.toml	Bumped django-lifecycle version constraint and uv override from 1.2.4 to 1.2.5
Hook decorator cleanup api/audit/models.py, api/environments/models.py, api/experimentation/models.py, api/features/feature_external_resources/models.py, api/features/models.py, api/features/release_pipelines/core/models.py, api/features/workflows/core/models.py, api/integrations/github/models.py, api/projects/models.py, api/users/models.py	Removed obsolete # type: ignore[misc] comments from @hook(...) decorators; minor kwargs formatting change in one Celery task call

Sequence Diagram(s)

Not applicable — this change consists solely of comment removals and a dependency version bump with no observable behavioural or flow changes.

Related issues: None referenced in the provided changes.

Related PRs: None referenced in the provided changes.

Suggested labels: dependencies, chore, type-check

Suggested reviewers: None specifically indicated by the diff content.

Poem:
A rabbit hopped through model files bright,
Snipping stale ignores left and right,
No logic bent, no tests to fright,
Just tidier hooks and a version's slight height,
django-lifecycle now shines 1.2.5 tonight. 🐇

---

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 9a90c458-5cd3-4344-a6ab-fce04c9e3b68

📥 Commits

Reviewing files that changed from the base of the PR and between e77c851 and bf2ae79.

📒 Files selected for processing (11)

• api/audit/models.py
• api/environments/models.py
• api/experimentation/models.py
• api/features/feature_external_resources/models.py
• api/features/models.py
• api/features/release_pipelines/core/models.py
• api/features/workflows/core/models.py
• api/integrations/github/models.py
• api/projects/models.py
• api/pyproject.toml
• api/users/models.py

[coderabbitai]

🗄️ Data Integrity & Integration | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
cd api
uv lock --check
rg -n '"django-lifecycle"' pyproject.toml uv.lock

Repository: Flagsmith/flagsmith

Length of output: 426

---

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n-- pyproject.toml (around the dependency) --\n'
sed -n '35,50p' api/pyproject.toml | cat -n

printf '\n-- uv.lock (matching entries) --\n'
sed -n '45,60p' api/uv.lock | cat -n
printf '\n'
sed -n '1524,1538p' api/uv.lock | cat -n

Repository: Flagsmith/flagsmith

Length of output: 3188

---

Refresh the lockfile for the dependency bump api/uv.lock still pins django-lifecycle==1.2.4, so it no longer matches the >=1.2.5,<1.3.0 constraint in api/pyproject.toml. Regenerate and commit the lockfile so locked installs pick up the updated version.