Skip to content

Add Error Assistant LLM integration#463

Open
priyavadan wants to merge 4 commits intomasterfrom
llm
Open

Add Error Assistant LLM integration#463
priyavadan wants to merge 4 commits intomasterfrom
llm

Conversation

@priyavadan
Copy link
Member

Origen::ErrorAssistant
applies mode precedence for off/forced/opt_in
falls back display name when friendly name missing
[WARNING] 0.070[0.070] || Error Assistant timed out after 0.01s
times out best effort and returns nil
passes site config settings into plugin context

[Error Assistant Response]
Based on the stack trace provided, here's my diagnosis:

## 1) Most likely root cause
This appears to be a test failure in the Error Assistant functionality within the Origen application's test suite. The exception is occurring at line 147 of the error_assistant_spec.rb file during RSpec test execution, suggesting either a test assertion failure or an unhandled exception within the Error Assistant feature being tested.

## 2) Suggested fix steps
• Check the specific test at line 147 in `spec/error_assistant_spec.rb` to understand what functionality is being tested
• Run the test with verbose output (`rspec --format documentation`) to see the actual error message and test description
• Verify that any dependencies or mock objects required by the Error Assistant are properly set up in the
  live test: raises and requests a real solution (manual)

Finished in 7.41 seconds (files took 0.39531 seconds to load)
5 examples, 0 failures

- Integrated OrigenLlm::ErrorAssistant for AI-powered error analysis
- Added site config options for LLM provider configuration
- Supports generic and Anthropic Claude API backends
- Privacy-by-default: disabled unless explicitly configured
- Best-effort timeout handling (default 3s)
- Comprehensive test coverage
@priyavadan priyavadan requested a review from a team February 21, 2026 19:36
Gemfile Outdated

# Required to run the concurrent test case patterns from OrigenSim
gem 'origen_jtag'
gem 'origen_llm', "~>0.1.2"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As a general question, is it worth adding origen_llm to the gemspec for application usage too? Or is this intended for just the core at this time?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fair call, good catch. It should be in the gemspec. I've updated it.

@pderouen
Copy link
Member

This is a nice feature! Any web docs update needed?

@priyavadan
Copy link
Member Author

This is a nice feature! Any web docs update needed?

I tried to sneak it in @pderouen ... yup ok, will add some docs

Copy link
Member

@pderouen pderouen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

love it!

Copy link

@ThaoHuynhFsl ThaoHuynhFsl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looking forward to it

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.

4 participants