Skip to content

[CLEAN] Synthetic Benchmark PR #30552 - feat(graph-engine): make layer runtime state non-null and bound early#176

Open
bar-qodo wants to merge 1 commit intobase_pr_30552_20260113_1188from
clean_pr_30552_20260113_1188
Open

[CLEAN] Synthetic Benchmark PR #30552 - feat(graph-engine): make layer runtime state non-null and bound early#176
bar-qodo wants to merge 1 commit intobase_pr_30552_20260113_1188from
clean_pr_30552_20260113_1188

Conversation

@bar-qodo
Copy link

Benchmark PR langgenius#30552

Type: Clean (correct implementation)

Original PR Title: feat(graph-engine): make layer runtime state non-null and bound early
Original PR Description: > [!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

Fixes langgenius#30551.

  • Introduce an uninitialized read-only runtime state and explicit error to keep GraphEngineLayer.graph_runtime_state non-null while surfacing early access.
  • Bind layer runtime context when registering layers and align built-in layers with the new contract.
  • Update docs and tests around layer initialization, and remove a truthiness guard that now raises under the new sentinel.

Screenshots

Before After
N/A N/A

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 dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

Original PR URL: langgenius#30552

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants