LearnNewsExamplesServices
Frontmatter
id8927
titleCleanup: Remove unused set_component mapping in AI Client
stateClosed
labels
airefactoring
assigneestobiu
createdAtJan 31, 2026, 5:46 PM
updatedAtJan 31, 2026, 5:51 PM
githubUrlhttps://github.com/neomjs/neo/issues/8927
authortobiu
commentsCount1
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtJan 31, 2026, 5:51 PM

Cleanup: Remove unused set_component mapping in AI Client

Closed v12.0.0 airefactoring
tobiu
tobiu commented on Jan 31, 2026, 5:46 PM

We identified that src/ai/Client.mjs maps set_component to ComponentService, but ComponentService lacks this method. This functionality is fully covered by InstanceService.setInstanceProperties.

Goal: Remove the mapping for set_component in src/ai/Client.mjs to clean up the API surface area.

Reference: src/ai/Client.mjs

me.serviceMap = {
    // ...
    set_component: component, // Remove this
    // ...
}
tobiu added the enhancement label on Jan 31, 2026, 5:46 PM
tobiu added the ai label on Jan 31, 2026, 5:46 PM
tobiu assigned to @tobiu on Jan 31, 2026, 5:46 PM
tobiu removed the enhancement label on Jan 31, 2026, 5:49 PM
tobiu added the refactoring label on Jan 31, 2026, 5:49 PM
tobiu changed title from Feat: Implement call_method tool for Neural Link to Cleanup: Remove unused set_component mapping in AI Client on Jan 31, 2026, 5:49 PM
tobiu referenced in commit fd73813 - "cleanup: Remove unused set_component mapping in AI Client (#8927)" on Jan 31, 2026, 5:51 PM
tobiu
tobiu Jan 31, 2026, 5:51 PM

Input from Gemini 3 Pro:

✦ I have removed the unused set_component mapping from src/ai/Client.mjs.

This cleanup reduces confusion as set_component was mapped to ComponentService but not implemented there, while set_instance_properties (mapped to InstanceService) covers the functionality.

tobiu closed this issue on Jan 31, 2026, 5:51 PM