feat(backend): Speed up graph create/update

[Pwuts]

• Resolves Fetch user credentials only once for on_graph_activate, on_graph_deactivate #10024

Caching the repeated DB calls by the graph lifecycle hooks significantly speeds up graph update/create calls with many authenticated blocks (~300ms saved per authenticated block)

Changes 🏗️

• Add and use IntegrationCredentialsManager.cached_getter(user_id) in lifecycle hooks
  • Split refresh_if_needed(..) method out of IntegrationCredentialsManager.get(..)
• Simplify interface of lifecycle hooks: change get_credentials parameter to user_id

Checklist 📋

For code changes:

• I have clearly listed my changes in the PR description
• I have made a test plan
• I have tested my changes according to the test plan:
  • Save a graph with nodes with credentials

[netlify]

✅ Deploy Preview for auto-gpt-docs-dev canceled.

Name	Link
🔨 Latest commit	676c268
🔍 Latest deploy log	https://app.netlify.com/projects/auto-gpt-docs-dev/deploys/683358cbecc95f000803a6f6

[netlify]

✅ Deploy Preview for auto-gpt-docs canceled.

Name	Link
🔨 Latest commit	676c268
🔍 Latest deploy log	https://app.netlify.com/projects/auto-gpt-docs/deploys/683358cb4e0adf0008f3dc5f

[deepsource-io]

Here's the code health analysis summary for commits dc981b5..676c268. View details on DeepSource ↗.

Analysis Summary

Analyzer	Status	Summary	Link
JavaScript	✅ Success		View Check ↗
Python	✅ Success	❗ 7 occurences introduced 🎯 7 occurences resolved	View Check ↗

---

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

[AutoGPT-Agent]

The PR addresses reducing DB calls for graph operations as stated in the title. The changes include adding a credentials caching function and refactoring the credentials manager. The description clearly explains the changes and references issue #10024. The author has marked that they've made a test plan but has not completed testing according to that plan, as the 'tested my changes' checkbox is unchecked. All changes pass the user_id correctly in the credentials manager functions, maintaining security. The PR title follows conventional commit format with the correct type and scope.

[AutoGPT-Agent]

The PR meets most of the requirements, but the checklist is not completely filled out. In the test plan section, there are checkboxes that remain unchecked, indicating that the author has not yet tested their changes according to the plan they defined. The PR follows proper title formatting with 'feat(backend)' as the type and scope. The description clearly explains the purpose (speeding up graph operations with authenticated blocks) and lists the specific changes made. The code changes appear to properly handle user_id in credential management, which was an important requirement. All changes appear to be within scope of the PR title.

[AutoGPT-Agent]

The PR meets most of the requirements. The title follows conventional commit format with the type 'feat' and scope 'backend'. The description clearly references issue #10024 and explains the intent of the changes (speeding up graph update/create calls with authenticated blocks). The changes listed in the PR description match what's shown in the diff - extracting a refresh_if_needed method and implementing a _cached_credentials_getter function. However, the checklist in the PR description is incomplete - the author has marked that they have a test plan but has not actually performed the testing according to that plan (the testing checkbox is unchecked). The changes to data/*.py files appear to properly handle user_id checking through the credential manager.

[Pwuts]

If you read this, it works :)

[kcze]

LGTM

[kcze]

LGTM!