Skip to content

Conversation

@PyContriver
Copy link
Collaborator


name: Pull request
about: Describe the proposed change

Before review

  • All tests are passing
  • Description mentions issue via
    (fixes|closes|related): #<issue_number|AAP-number>
  • PR is kept as draft until all checks above are met

Review checklist

  • Size of change. If it can be split into smaller PRs, please do so.
  • Docs checks:
    • heading titles are short and render without wrapping in sidebar
    • code blocks and diagrams render correctly in dark/light mode
    • images and videos are not blurry, small in size and use appropriate
      format such as png, svg, webp, mp4 (not gif).
    • Less is more, no boilerplate language. Feel free to use AI to help with
      this and with correct grammar.

@github-actions
Copy link
Contributor

Label error. Requires exactly 1 of: breaking, build, chore, ci, docs, feat, fix, perf, refactor, style, test. Found: . Follow https://www.conventionalcommits.org to get auto-labeling to work correctly.

ganeshrn and others added 12 commits November 20, 2025 01:48
* Updated lightspeed setting to read LLM releated configuration
  - Model Name
  - API endpoint
  - Provider name
*  Add google, openai and custom LLM provider for playbook generation,
   playbook explanation and role generation lightspeed functionality
*  Add telemetry events for LLM providers to send the events from
   the extension directly to segement in case proivder is not WCA
   if the telemetry setting is enabled by the user.
- Remove extra blank line in test/helper.ts
- Simplify type assertion in factory.test.ts
- Add eslint-disable for @typescript-eslint/no-explicit-any in test files
- Remove unused error variables in catch blocks
- Tests use dynamic imports and mocks requiring any types
- Add 'genai' to cspell dictionary
- Apply biome formatter fixes to package.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants