[agent/claude] Fix ide/main.go to compile with current Core and MCP APIs. T... #2
No reviewers
Labels
No labels
needs-review
needs-review
athena
athena-gemini
audit
clotho
clotho-gemini
codex
darbs-claude
security
wiki
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: core/ide#2
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "agent/fix-ide-main-go-to-compile-with-current"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Task
Fix ide/main.go to compile with current Core and MCP APIs. The old patterns need replacing: (1) core.WithName("name", factory) → create Core with core.New(), call factory directly, register via c.Service("name", core.Service{...}). (2) mcp.WithWorkspaceRoot/WithWSHub/WithSubsystem → mcp.New(mcp.Options{WorkspaceRoot: cwd, WSHub: hub, Subsystems: []mcp.Subsystem{...}}). (3) core.ServiceForT is removed — keep a direct reference to the service from when you create it. (4) Import paths may need updating from forge.lthn.ai to dappco.re if go.mod uses dappco.re. Check with: go build ./...
Agent: claude
Commits: 1
Branch:
agent/fix-ide-main-go-to-compile-with-currentAuto-created by core-agent dispatch system.
Co-Authored-By: Virgil virgil@lethean.io
Replace deprecated patterns with current API conventions: - core.WithName/WithService → core.New() + RegisterService + RegisterAction - mcp.WithWorkspaceRoot/WithWSHub/WithSubsystem → mcp.New(mcp.Options{}) - core.ServiceFor[T] → direct service reference - Add cancellable context and ServiceShutdown to GUI mode - Handle previously-discarded errors from api.New and config.New Co-Authored-By: Virgil <virgil@lethean.io>Tests Passed — Merge Failed
go test ./...passed but merge failed: forgeMergePR: HTTP 405: Please try again laterNeeds Review
Merge conflict persists after rebase. Auto-merge gave up after retry.
Labelled
needs-reviewfor human attention.Pull request closed