Skip to content

refactor(dgw): give the gateway state the provisioner task store - #2042

Closed
irvingouj@Devolutions (irvingoujAtDevolution) wants to merge 1 commit into
feat/provisioner-taskfrom
feat/ai-analysis-plumbing
Closed

refactor(dgw): give the gateway state the provisioner task store#2042
irvingouj@Devolutions (irvingoujAtDevolution) wants to merge 1 commit into
feat/provisioner-taskfrom
feat/ai-analysis-plumbing

Conversation

@irvingoujAtDevolution

@irvingoujAtDevolution irvingouj@Devolutions (irvingoujAtDevolution) commented Oct 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Plumbing only, no behavior change:

  • DgwState gets the provisioner task store, on its own gateway.db connection.
  • DgwState::mock becomes async (it opens an in-memory store), so a few test call sites gain .await.
  • job_queue::MAX_ATTEMPTS (5, same as the current default) is passed to the job queue explicitly, so the AI analysis job can't drift from it.

🤖 Generated with Claude Code

@irvingoujAtDevolution
irvingouj@Devolutions (irvingoujAtDevolution) removed this pull request from stack #2045 October 6, 2026 21:26
@irvingoujAtDevolution @claude
refactor(dgw): give the gateway state the provisioner task store
370cccb 
The Gateway state now holds the provisioner task store, on its own
gateway.db connection, so the work it records can reach it. Building
the mock state opens an in-memory database, so `DgwState::mock` is now
async.

The job queue attempt limit is now the named constant `MAX_ATTEMPTS`,
still 5, so a job can refer to it.

No behavior change.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@irvingoujAtDevolution
irvingouj@Devolutions (irvingoujAtDevolution) removed this pull request from stack #2046 October 7, 2026 20:36
@irvingoujAtDevolution

irvingouj@Devolutions (irvingoujAtDevolution) commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

Folded into #2044 as its first commit: the store is now added to DgwState in the same PR that first uses it. The stack is now #2039 -> #2041 -> #2043 -> #2044 -> #2040.

Note

LLM-assisted content (no human feedback).

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@irvingoujAtDevolution