Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 6, 2026

Multiple early exit paths across the E-Document AI tools silently return without any telemetry, providing zero visibility into why AI matching produces no results for users.

  • E-Doc. AI Tool Processor: Added Session.LogMessage with Verbosity::Warning at both early exit paths in Setup (capability not registered, capability not active). Moved AISystem := EDocAISystem assignment before capability checks so the feature name is available for telemetry context.
  • E-Doc. GL Account Matching: Added telemetry when no G/L Accounts exist in the system (GLAccount.IsEmpty()), which causes the tool to skip AI matching entirely.
  • E-Doc. Deferral Matching: Added telemetry when no Deferral Templates exist in the system (DeferralTemplate.IsEmpty()), which causes the tool to skip AI matching entirely.
  • E-Doc. Historical Matching: Added telemetry in PrepareHistoricalData when no E-Document Purchase Lines are found and when no potential historical matches are collected after data collection.

All telemetry tags left as '' per convention (filled by automated script). All use the existing Session.LogMessage pattern with Verbosity::Warning.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: Groenbech96 <17690329+Groenbech96@users.noreply.github.com>
Copilot AI changed the title [WIP] Add telemetry for AI Tools in e-documents core app Add telemetry to E-Document AI Tool Processor Setup early exits Feb 6, 2026
Copilot AI requested a review from Groenbech96 February 6, 2026 16:41
@Groenbech96
Copy link
Contributor

@copilot Its a good start. If you look at the specific tools, analyze if we are potentially missing other things.

…torical Matching

Co-authored-by: Groenbech96 <17690329+Groenbech96@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 6, 2026

@copilot Its a good start. If you look at the specific tools, analyze if we are potentially missing other things.

Added telemetry to the early exit paths in each specific AI tool (9059a23):

  • GL Account Matching: logs when GLAccount.IsEmpty() — no GL accounts in the system
  • Deferral Matching: logs when DeferralTemplate.IsEmpty() — no deferral templates in the system
  • Historical Matching: logs when no purchase lines are found, and when no potential matches are collected after data collection

All use Session.LogMessage with Verbosity::Warning and tags left as '' per convention.

Copilot AI changed the title Add telemetry to E-Document AI Tool Processor Setup early exits Add telemetry to E-Document AI Tools early exits Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants