Ean Schuessler / mo-mcp

Use SystemMessageType format: systemMessageTypeId instead of enumTypeId/enumTypeId

Ean Schuessler committed 2026-02-04 19:08:17 -0600

Create separate enum data file for agent runtime
- AiServiceType enum type and values
- SmtyAgentTask enum for SystemMessageType
Update component.xml to load AgentEnumData.xml

Ean Schuessler committed 2026-02-04 19:05:45 -0600

enabled and description are not valid fields for UserAccount entity

Ean Schuessler committed 2026-02-04 18:51:17 -0600

Remove ProductStoreAiConfig and enumerations that have FK issues
Keep only essential data: AGENT_CLAUDE user, AgentUsers group, permissions
Will add AI config after verifying basic infrastructure loads

Ean Schuessler committed 2026-02-04 18:47:44 -0600

POPCOMMERCE_RETAIL doesn't exist - use POPC_DEFAULT instead

Ean Schuessler committed 2026-02-04 18:42:17 -0600

ScheduledJob cannot be loaded via seed data - needs to be created through service
Will create AgentQueuePoller via service call after rebuild

Ean Schuessler committed 2026-02-04 18:39:05 -0600

Remove package prefixes from entity names to match data loader format
- ScheduledJob instead of moqui.service.ScheduledJob
- UserAccount instead of moqui.security.UserAccount
- UserGroup instead of moqui.security.UserGroup
- UserGroupMember instead of moqui.security.UserGroupMember
- ArtifactGroup instead of moqui.security.ArtifactGroup
- ArtifactGroupMember instead of moqui.security.ArtifactGroupMember
- ArtifactAuthz instead of moqui.security.ArtifactAuthz
- ProductStoreAiConfig instead of moqui.mcp.agent.ProductStoreAiConfig

Ean Schuessler committed 2026-02-04 18:33:29 -0600

- Use crunchy.private.brainfood.com:11434/v1 endpoint
- Model: bf-ai
- API key: brainfood

Ean Schuessler committed 2026-02-04 17:38:10 -0600

- AgentEntities.xml: Entity extensions for agent tasks
- AgentServices.xml: Agent runner, client, and queue services
- AgentData.xml: Scheduled job, user, and AI config seed data

Ean Schuessler committed 2026-02-04 17:25:37 -0600

Ean Schuessler committed 2026-01-30 13:22:36 -0600

Introduce clean adapter layer between Moqui infrastructure and MCP protocol:

- transport/MoquiMcpTransport: Interface abstracting transport concerns
- transport/SseTransport: SSE implementation with session management
- adapter/McpSessionAdapter: Maps Moqui Visit to MCP sessions
- adapter/McpToolAdapter: Maps MCP tools/methods to Moqui services
- adapter/MoquiNotificationMcpBridge: Bridges Moqui notifications to MCP

Simplify EnhancedMcpServlet to orchestrator role, removing inline session
management, SSE logic, and tool dispatch. Remove redundant session
validation in Initialize service (MoquiAuthFilter handles auth).

Delete obsolete files:
- VisitBasedMcpSession.groovy (replaced by McpSessionAdapter)
- JsonRpcMessage.groovy (using plain Maps)
- MoquiMcpTransport.groovy (replaced by new interface)

Ean Schuessler committed 2026-01-28 23:20:57 -0600

- Index business process workflows from BUSINESS_PROCESSES wiki space
- Add workflow support to search and help tools
- Improve documentation with better organized help content
- Add type attributes to seed data files
- Enhance error handling for workflow loading

Ean Schuessler committed 2026-01-28 13:24:47 -0600

- Fix column extraction to include fields with both search and display widgets
- Add all ID fields to row ref as pipe-delimited key=value pairs for unambiguous LLM parsing
- Show NULL values as '(none)' in grid cells for consistency
- Add ARIA-compliant action result feedback (role=alert for errors, role=status for success)
- Pass actionResult through BrowseScreens ARIA mode
- Hide batch_operations tool until fixed
- Default renderMode to 'aria' instead of 'compact'

Ean Schuessler committed 2026-01-23 13:08:43 -0600

When pseudoId differs from the internal productId (e.g., DEMO_VAR_GR_SM_COPY
vs 100204), models were using the display ID for service calls causing FK errors.

Now extracts the actual entity ID from the row's link URL and includes it in
the row data:
- compact: {id: 'DEMO_VAR_GR_SM_COPY', productId: '100204', link: '...'}
- aria: {ref: 'DEMO_VAR_GR_SM_COPY', productId: '100204', ...}

This is a generic solution that works for any entity type by parsing the
primary key parameter from the Edit link URL.

Fixes QA issue where models used pseudoId for applyProductFeatures causing
'record does not exist' referential integrity errors.

Ean Schuessler committed 2026-01-21 19:56:59 -0600

- FTL: Capture Moqui pagination vars (pageIndex, pageSize, pageMaxIndex, listCount)
- Compact mode: Add nextCursor object with pageIndex, pageSize, hasMore
- ARIA mode: Add nextCursor to grid nodes
- Add totalCount to show full result size

Usage: Pass nextCursor values as parameters to get next page:
  parameters: {pageIndex: 1, pageSize: 20}

Response format:
  nextCursor: {pageIndex: 1, pageSize: 20, hasMore: true}
  totalCount: 29

Ean Schuessler committed 2026-01-21 15:37:43 -0600

- FTL: Detect header-field search widgets (text-find, date-find, etc.)
- FTL: Add 'searchable' flag and search-specific types to field metadata
- ARIA: Add role='search' landmark before grids with searchable fields
- Compact: Add searchParams array listing filterable parameters
- Filter out 'get*' transitions (autocomplete helpers) from transitions list

This helps LLM clients understand how to filter form-list results by
passing parameters, rather than trying to call autocomplete transitions.

Ean Schuessler committed 2026-01-21 15:16:23 -0600

XML mode renders nearly empty screen hierarchy (just path structure, no content
or data), which isn't useful for MCP clients.

Remaining render modes:
- compact: structured actionable summary (default)
- aria: accessibility tree for field discovery
- text: plain text rendering
- html: full HTML rendering

Ean Schuessler committed 2026-01-21 14:50:44 -0600

These modes render Vue.js wrapper templates for web frontends, not actual
screen content. They only show outer container HTML with subscreen placeholders,
which isn't useful for MCP clients.

Remaining render modes:
- compact: structured actionable summary (default)
- aria: accessibility tree for field discovery
- text: plain text rendering
- html: full HTML rendering
- xml: XML rendering

Ean Schuessler committed 2026-01-21 14:48:21 -0600

The mcp render mode exposed too much internal detail (service names, full
metadata) which confused smaller models into trying to call services directly.

Changes:
- Remove 'mcp' from renderMode descriptions in tool definitions
- Map renderMode='mcp' to 'compact' if explicitly requested
- Update AGENTS.md to remove mcp mode documentation
- Internal rendering still uses mcp mode, just not exposed to users

Users should use:
- compact (default): actionable summary
- aria: accessibility tree for field discovery
- text/html: rendered output

Ean Schuessler committed 2026-01-21 14:46:19 -0600

Adds clear instructions in root wiki doc explaining:
- How to execute actions (use action parameter, NOT service calls)
- How to get dropdown options (use moqui_get_screen_details)
- How to get action documentation (use moqui_get_help with help URI)
- Updated screen paths to use apps/PopcAdmin format
- Added global navigation paths (Messages, Tasks, Calendar)

This helps models understand how to use the system after we removed
service names and dropdown examples from responses.

Ean Schuessler committed 2026-01-21 13:08:09 -0600

ARIA and compact modes now cleaner for smaller models:

- Remove service names from actions (confuses models into trying to call services)
- Rename 'describedby' to 'help' for clarity (points to moqui_get_help tool)
- Remove wiki:screen references (not useful, wiki content already in response)
- Fix grid row refs to use specific IDs (productFeatureId, toProductId) instead
  of repeating generic productId for all rows
- Filter navigation links: remove cross-app links, remove action URLs with
  encoded timestamps (fromDate/thruDate parameters)
- Remove dropdown options counts and random examples - just indicate type,
  use moqui_get_screen_details for actual options

This significantly reduces context volume and eliminates patterns that
caused smaller models to hallucinate service calls.

Ean Schuessler committed 2026-01-21 13:03:58 -0600

ARIA mode was truncating grid rows to 3, links to 10, and actions to 15.
This caused models to miss important data (e.g., only seeing 3 of 8 features
or 3 of 16 variant associations).

Now ARIA mode shows complete data like text mode does.

Ean Schuessler committed 2026-01-21 11:45:29 -0600

Adds globalNav array to every browse_screens response with links to:
- My Notifications (apps/my/User/Notifications)
- My Messages (apps/my/User/Messages/FindMessage)
- My Calendar (apps/my/User/Calendar/MyCalendar)
- My Tasks (apps/my/User/Task/MyTasks)

These mirror the MyAccountNav component in the web UI and are always
available regardless of which app the user is currently browsing.

Also adds a hint in uiNarrative.notes about globalNav and search.

Ean Schuessler committed 2026-01-20 23:51:44 -0600

The username parameter was being injected into screen render params, which
conflicted with FindCustomer's 'username' search field. This caused the
screen to filter results to only show customers with username matching
the current user (john.sales*), returning 0 results.

User context is already available via ec.user for authorization purposes,
so injecting these params was redundant and harmful.

Ean Schuessler committed 2026-01-20 23:24:08 -0600

Critical fix: Errors now correctly captured and reported
- Capture errors immediately after render() in CustomScreenTestImpl
- Added saveMessagesToSession() to WebFacadeStub for framework compatibility
- Actions that fail validation now report status:'error' instead of false success

Documentation fix: Clarify two different feature services
- applyProductFeatures: Apply EXISTING features to products (most common)
- createProductFeature: Create NEW feature definitions (rare)
- Updated EditProduct and ProductFeature wiki docs with correct examples

Ean Schuessler committed 2026-01-20 22:52:24 -0600

serverInfo now includes:
- hostname and ipAddress for server identification
- javaVersion and javaVendor for environment info
- serverStartTime (epoch ms) for uptime calculation
- moquiVersion, runtimePath for deployment verification
- currentUser/currentUserId for auth confirmation

Follows MCP standard - info returned via initialize response

Ean Schuessler committed 2026-01-20 22:01:33 -0600

Critical fixes:
- Set resolvedScreenDef after successful literal path resolution (was falling back to webroot)
- Traverse default subscreens when looking up transitions (matching action discovery logic)
- Build full path including subscreens for framework transition execution

New features:
- moqui_batch_operations tool for multi-step workflows
- Enhanced validation error handling with field-level details
- ARIA mode now includes value, ref, description, describedby attributes

Documentation:
- Added render modes table and examples to AGENTS.md
- Updated README with MARIA format explanation
- Added MCP_SERVICE_DOCS wiki space with service parameter docs

Ean Schuessler committed 2026-01-20 20:20:48 -0600

When looking up wiki docs for a screen, now walks up the path hierarchy
to find the most specific match. For example, for path
'Catalog/Product/EditProduct/Assocs', it tries:

1. Catalog/Product/EditProduct/Assocs (most specific)
2. Catalog/Product/EditProduct (inherits if no specific doc)
3. Catalog/Product
4. Catalog

This allows parent screen docs to apply to all child tabs/subscreens
unless overridden with more specific documentation.

Updated both getWikiInstructions (used by execute#ScreenAsMcpTool) and
loadWikiContent (used by moqui_browse_screens) to use this approach.

Ean Schuessler committed 2026-01-19 21:26:21 -0600

Clarifies that User/Messages is a container and FindMessage should be
used for full functionality (actions, transitions).

Ean Schuessler committed 2026-01-19 20:56:33 -0600

Shows screen-transition actions like editMessage, messageThread in a
'transitions' array. Filters out internal actions (formSelectColumns,
formSaveFind, screenDoc, actions).

Ean Schuessler committed 2026-01-19 20:49:59 -0600

- Updated PartyMessages wiki docs to emphasize partyId is required
- Added missingRequired and error fields to compact format when
  required parameters are missing
- This helps LLMs understand why a screen shows no forms/grids

Ean Schuessler committed 2026-01-19 20:44:31 -0600

Added MCP screen documentation for:
- FindParty: Quick actions for creating people, orgs, accounts, employees
- EditParty: Party detail management, roles, classifications, user accounts
- PartyMessages/FindMessage: Internal messaging between parties
- FindProduct: Product search and creation, variant workflow overview
- EditProduct: Features, associations, complete variant setup guide

Each doc includes example action calls with parameters.

Ean Schuessler committed 2026-01-19 20:31:46 -0600

- Add compact render mode (now default): LLM-optimized ~75% smaller output
  - Shows forms with fields, grids with sample rows, actions with services
  - Truncates dropdown options with fetch hints

- Add ARIA render mode: W3C accessibility tree format
  - Maps Moqui fields to ARIA roles (textbox, combobox, checkbox, etc.)

- Fix action execution to use framework transitions
  - Actions now append to screen path for proper transition handling
  - Framework handles inheritance, pre/post actions, response handling

- Fix CSRF bypass for MCP requests
  - Set moqui.request.authenticated=true in MockHttpServletRequest attributes

Ean Schuessler committed 2026-01-19 20:29:33 -0600

- Remove remaining isTerse references from UiNarrativeBuilder
- Truncate dropdown options to 10 in browse_screens with fetchHint
- get_screen_details returns full options via mcpFullOptions flag
- Extract wiki summary as top-level field in responses
- Add extractSummary helper for first non-header paragraph

Ean Schuessler committed 2026-01-19 19:32:33 -0600

- Fix sri.getFieldOptions() call to pass drop-down node instead of parent field node
- Remove terse parameter from all services and serialization
- Increase safety limits by 10x: lists 10K items, strings 1MB, unknown types 10K chars
- Add form-list field metadata extraction in FTL macros
- Simplify McpFieldOptionsService to use CustomScreenTestImpl approach
- Clean up redundant _fixed.groovy file

Ean Schuessler committed 2026-01-19 15:46:51 -0600

- Extract service name and in-map from transition XML
- Capture depends-on parameter attribute for field name mapping
- Parse in-map parameter mapping for proper service calls
- Support server-search autocomplete with term parameter
- Fallback to ScreenAsMcpTool for entity-find transitions
- Enhanced response format detection and error handling

This implements TODO 1-3 from AGENTS.md:
- TODO 1: Capture Depends-on Parameter Attribute
- TODO 2: Extract Transition Service Name
- TODO 3: Parse in-map Parameter Mapping

Files modified:
- DefaultScreenMacros.mcp.ftl: Extract transition metadata during screen render
- McpFieldOptionsService.groovy: Use extracted metadata for intelligent autocomplete

Ean Schuessler committed 2026-01-15 11:57:20 -0600

Ean Schuessler committed 2026-01-13 18:36:41 -0600

Error Handling:
- Check ec.message.hasError() before and after transaction flush
- Return "status": "error" with error messages instead of false success
- Prevents silent failures when constraint violations occur (e.g., invalid Party IDs)

Dropdown Metadata:
- Capture static dropdown options from sri.getFieldOptions()
- Capture dynamic-options config (transition, serverSearch, minLength, parameterMap)
- Enables model to understand searchable dropdowns vs static lists

This improves the model's ability to:
1. See clear error messages when operations fail
2. Understand dropdown behavior (server search vs static)
3. Take corrective action based on error feedback

Ean Schuessler committed 2026-01-13 18:29:36 -0600

Fix path references in data files to use forward slash notation:
- /PopCommerce/PopCommerceAdmin.Catalog.Product → /PopCommerce/PopCommerceAdmin/Catalog/Product
- /PopCommerce/PopCommerceAdmin.Order.FindOrder → /PopCommerce/PopCommerceAdmin/Order/FindOrder
- /PopCommerce/PopCommerceRoot.Customer → /PopCommerce/PopCommerceRoot/Customer

Also updates:
- AGENTS.md: Remove completed path delimiter test items
- Fix wiki page path references (SimpleScreens.Root → SimpleScreens/Root)

Aligns with slash-based path system adopted in earlier refactor.

Ean Schuessler committed 2026-01-13 15:10:30 -0600

Ean Schuessler committed 2026-01-13 13:32:19 -0600