Skip to content

Conversation

@kaushiki069
Copy link
Collaborator

@kaushiki069 kaushiki069 commented Dec 3, 2025

The intent of the PR is to add the further configuration components and webview related changes to support BYOLLM (Bring Your Own Large Language Model). This proposes subsequent changes over the foundational Gemini SDK integration component [https://github.com//pull/2329]. It also entails addition of - unit tests and UI tests around the new changes being proposed, updates to the existing UI tests owing to the changes in the webviews and vsix file limit increase.

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.

PyContriver and others added 30 commits November 25, 2025 11:14
- Add outlineGenerator utility for playbook/role outline generation
- Add Google/WCA API endpoint constants to lightspeed.ts
- Add GOOGLE_DEFAULT_MODEL constant
- Add getLightspeedLogger singleton function to logger.ts

These are required dependencies for the Google provider implementation.
- Add GOOGLE_API_ENDPOINT constant
- Add WCA_API_ENDPOINT_DEFAULT constant
- Add GOOGLE_DEFAULT_MODEL constant

Required by factory.ts for provider configuration.
- Add ansibleContext.ts for Ansible-specific context processing
- Add js-yaml ^4.1.0 dependency (used by ansibleContext.ts)
- Add @types/js-yaml ^4.0.9 for TypeScript types
- Required by base.ts applyAnsibleContext() and cleanAnsibleOutput() methods
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants