langgenius/dify

[Chore/Refactor] db.session.xxx -> with Session as session

Chiusa

#24.115 aperta il 18 ago 2025

 (12 commenti) (1 reazione) (0 assegnatari)TypeScript (22.210 fork)batch import
good first issue

Metriche repository

Star
 (141.412 stelle)
Metriche merge PR
 (Merge medio 1g 21h) (579 PR mergiate in 30 g)

Descrizione

Self Checks

  • I have read the Contributing Guide and Language Policy.
  • This is only for refactoring, if you would like to ask a question, please head to Discussions.
  • I have searched for existing issues search for existing issues, including closed ones.
  • I confirm that I am using English to submit this report, otherwise it will be closed.
  • 【中文用户 & Non English User】请使用英语提交,否则会被关闭 :)
  • Please do not modify this template :) and fill in all the required fields.

Description

db.session.commit() is Flask-SQLAlchemy

with Session(...) is Pure SQLAlchemy

Motivation

for migration.

Additional Context

https://docs.sqlalchemy.org/en/20/orm/session_basics.html#framing-out-a-begin-commit-rollback-block

part of COM-7832

need https://github.com/facebook/pyrefly/issues/954

one example: #24135

Guida contributor