Skip to content

Commit f0d0f9a

Browse files
Clean up TASK_QUEUE: remove completed and superseded
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 7cb1dbc commit f0d0f9a

File tree

1 file changed

+0
-104
lines changed

1 file changed

+0
-104
lines changed

TASK_QUEUE.json

Lines changed: 0 additions & 104 deletions
Original file line numberDiff line numberDiff line change
@@ -11,16 +11,6 @@
1111
"status": "blocked",
1212
"notes": "Blocked on SDK release. Not actionable yet."
1313
},
14-
{
15-
"id": "T15",
16-
"title": "Change default tool execution to run in workflow context",
17-
"description": "Currently unannotated tools passed to defaultTools() are rejected. Change so they execute directly in workflow context by default \u2014 user is responsible for determinism. Remove @DeterministicTool annotation (no longer needed since direct execution is the default). Remove SandboxingAdvisor, LocalActivityToolCallbackWrapper, ExecuteToolLocalActivity, and ExecuteToolLocalActivityImpl. Remove ExecuteToolLocalActivityImpl registration from SpringAiPlugin. Keep @SideEffectTool as a convenience for wrapping in Workflow.sideEffect(). Keep activity stub / nexus stub auto-detection as shortcuts.",
18-
"severity": "high",
19-
"category": "refactor",
20-
"depends_on": [],
21-
"status": "completed",
22-
"notes": "Blocked on PR review discussion with tconley1428. Agreed on direction but need to finalize details before implementing. Proposed design: https://github.com/temporalio/sdk-java/pull/2829#discussion_r3060711651"
23-
},
2414
{
2515
"id": "T22",
2616
"title": "Discuss: temporal-spring-ai-starter artifact for easier onboarding",
@@ -30,100 +20,6 @@
3020
"depends_on": [],
3121
"status": "blocked",
3222
"notes": "DABH review comment: https://github.com/temporalio/sdk-java/pull/2829/files#r3053808755. Do after merging the main PR \u2014 not urgent for initial landing."
33-
},
34-
{
35-
"id": "T23",
36-
"title": "Discuss: ActivityMcpClient capability caching and replay",
37-
"description": "ActivityMcpClient caches getServerCapabilities() after first call. DABH asks if stale cache is a replay concern. Probably fine (activity result is in history, so replay uses the original value \u2014 which is correct). But worth confirming the design intent.",
38-
"severity": "low",
39-
"category": "discussion",
40-
"depends_on": [],
41-
"status": "completed",
42-
"notes": "Replied to DABH: cache prevents non-determinism (live vs replay would diverge without it). Standard MCP practice."
43-
},
44-
{
45-
"id": "T24",
46-
"title": "Discuss: Change ChatModelTypes.rawContent from Object to String",
47-
"description": "Change ChatModelTypes.Message rawContent from Object to String. Spring AI's Content.getText() returns String. We always cast to String on both sides anyway. Object type gives false flexibility that would ClassCastException at runtime.",
48-
"severity": "low",
49-
"category": "bugfix",
50-
"depends_on": [],
51-
"status": "completed",
52-
"notes": "DABH review comment: https://github.com/temporalio/sdk-java/pull/2829/files#r3054049714. Verified Spring AI Message interface uses String, not Object."
53-
},
54-
{
55-
"id": "T25",
56-
"title": "Reply: compatibility matrix in docs",
57-
"description": "DABH suggests documenting the compatibility matrix (Java version, Spring Boot version, Spring AI version). Acknowledge and defer to a docs PR.",
58-
"severity": "low",
59-
"category": "reply",
60-
"depends_on": [],
61-
"status": "completed",
62-
"notes": "Replied to DABH acknowledging. Follow-up task T29 created."
63-
},
64-
{
65-
"id": "T26",
66-
"title": "Reply: SandboxingAdvisor lacks tests",
67-
"description": "DABH notes SandboxingAdvisor has no tests. Likely moot if T15 removes it. Reply explaining that.",
68-
"severity": "low",
69-
"category": "reply",
70-
"depends_on": [
71-
"T15"
72-
],
73-
"status": "superseded",
74-
"notes": "DABH review comment: https://github.com/temporalio/sdk-java/pull/2829/files#r3053836427 Moot once T15 removes these classes."
75-
},
76-
{
77-
"id": "T27",
78-
"title": "Reply: ToolContext silently dropped in LocalActivityToolCallbackWrapper",
79-
"description": "DABH notes ToolContext is silently ignored. Likely moot if T15 removes LocalActivityToolCallbackWrapper. Reply explaining that.",
80-
"severity": "low",
81-
"category": "reply",
82-
"depends_on": [
83-
"T15"
84-
],
85-
"status": "superseded",
86-
"notes": "DABH review comment: https://github.com/temporalio/sdk-java/pull/2829/files#r3054036773 Moot once T15 removes these classes."
87-
},
88-
{
89-
"id": "T28",
90-
"title": "Restore VectorStorePlugin and EmbeddingModelPlugin as public classes",
91-
"description": "Restore the plugin subclasses so users not using auto-config can create them manually (e.g. new VectorStorePlugin(vectorStore)). Auto-config uses them too. Reverts the builder-inline approach from the earlier refactor.",
92-
"severity": "medium",
93-
"category": "refactor",
94-
"depends_on": [],
95-
"status": "completed",
96-
"notes": "tconley1428 review comments on SpringAiPlugin.java and SpringAiVectorStoreAutoConfiguration.java"
97-
},
98-
{
99-
"id": "T29",
100-
"title": "Add README with compatibility matrix to temporal-spring-ai module",
101-
"description": "Document supported versions: Java 17+, Spring Boot 3.x+, Spring AI 1.1.0, Temporal SDK 1.33.0+.",
102-
"severity": "high",
103-
"category": "docs",
104-
"depends_on": [],
105-
"status": "completed",
106-
"notes": "Do in this PR, not a follow-up."
107-
},
108-
{
109-
"id": "T30",
110-
"title": "Fix CI: Edge build fails with Java version mismatch",
111-
"description": "Edge CI sets edgeDepsTest which compiles temporal-sdk at Java 21. Our module hardcodes Java 17, causing Gradle to reject the dependency. Fix: use 21 when edgeDepsTest is set, 17 otherwise.",
112-
"severity": "high",
113-
"category": "bugfix",
114-
"depends_on": [],
115-
"status": "completed",
116-
"notes": "Edge CI log showed: Dependency resolution is looking for a library compatible with JVM runtime version 17, but temporal-sdk is only compatible with JVM runtime version 21 or newer."
117-
},
118-
{
119-
"id": "T31",
120-
"title": "Fix CI: Docker build fails with Java 11 (release 17 not supported)",
121-
"description": "Docker CI runs Java 11 which cannot compile --release 17. Conditionally exclude temporal-spring-ai from settings.gradle and BOM when JDK < 17.",
122-
"severity": "high",
123-
"category": "bugfix",
124-
"depends_on": [],
125-
"status": "completed",
126-
"notes": "Docker CI log showed: error: release version 17 not supported. JAVA_HOME was Java 11."
12723
}
12824
]
12925
}

0 commit comments

Comments
 (0)