Skip to content

fix: Working outside of application context.#38300

Merged
hjlarry merged 1 commit into
langgenius:mainfrom
fatelei:issue-38299
Jul 2, 2026
Merged

fix: Working outside of application context.#38300
hjlarry merged 1 commit into
langgenius:mainfrom
fatelei:issue-38299

Conversation

@fatelei

@fatelei fatelei commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Important

  1. Make sure you have read our contribution guidelines
  2. Ensure there is an associated issue and you have been assigned to it
  3. Use the correct syntax to link this PR: Fixes #<issue number>.

Summary

fix #38299

Screenshots

Before After
... ...

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran make lint && make type-check (backend) and cd web && pnpm exec vp staged (frontend) to appease the lint gods

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jul 2, 2026
hjlarry
hjlarry approved these changes Jul 2, 2026
View reviewed changes

@hjlarry hjlarry left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jul 2, 2026
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Pyrefly Type Coverage

Metric Base PR Delta
Type coverage 51.85% 51.85% +0.00%
Strict coverage 51.37% 51.37% +0.00%
Typed symbols 31,521 31,523 +2
Untyped symbols 29,543 29,543 0
Modules 2951 2951 0

@hjlarry hjlarry enabled auto-merge July 2, 2026 06:35
@hjlarry hjlarry added this pull request to the merge queue Jul 2, 2026
Merged via the queue into langgenius:main with commit 826b259 Jul 2, 2026
31 checks passed
hjlarry pushed a commit to hjlarry/dify that referenced this pull request Jul 2, 2026
@fatelei @hjlarry
fix: Working outside of application context. (langgenius#38300)
8b51fba 
(cherry picked from commit 826b259)
hjlarry pushed a commit that referenced this pull request Jul 2, 2026
@fatelei @hjlarry
fix: Working outside of application context. (#38300)
30c8e9b 
(cherry picked from commit 826b259)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hjlarry hjlarry hjlarry approved these changes
@laipz8200 laipz8200 Awaiting requested review from laipz8200 laipz8200 is a code owner
@QuantumGhost QuantumGhost Awaiting requested review from QuantumGhost QuantumGhost is a code owner

Assignees

No one assigned

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Working outside of application context.

2 participants

@fatelei @hjlarry