Skip to content

fix(pi-continuous-learning): support custom analyzer providers#154

Merged
MattDevy merged 5 commits into
mainfrom
fix/pi-cl-custom-providers
Jul 17, 2026
Merged

fix(pi-continuous-learning): support custom analyzer providers#154
MattDevy merged 5 commits into
mainfrom
fix/pi-cl-custom-providers

Conversation

@MattDevy

Copy link
Copy Markdown
Owner

Summary

  • resolve analyzer models through Pi’s ModelRegistry
  • load custom providers, credentials, and request headers from ~/.pi/agent/models.json
  • use the registry for both normal analysis and consolidation
  • add regression coverage and update the relevant documentation

Testing

  • env HOME=/private/tmp/pi-extensions-test-home npm run check
  • npm run build -w packages/pi-continuous-learning

Closes #134

@github-actions

github-actions Bot commented Jul 17, 2026

Copy link
Copy Markdown

⚠️MegaLinter analysis: Success with warnings

Descriptor Linter Files Fixed Errors Warnings Elapsed time
⚠️ MARKDOWN markdownlint 2 9 0 0.66s
✅ REPOSITORY gitleaks yes no no 0.84s
✅ REPOSITORY git_diff yes no no 0.19s

Detailed Issues

⚠️ MARKDOWN / markdownlint - 9 errors
packages/pi-continuous-learning/README.md:1 MD041/first-line-heading/first-line-h1 First line in a file should be a top-level heading [Context: "![pi-continuous-learning banne..."]
packages/pi-continuous-learning/README.md:77 MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
packages/pi-continuous-learning/README.md:168 MD040/fenced-code-language Fenced code blocks should have a language specified [Context: "```"]
packages/pi-continuous-learning/README.md:211:20 MD049/emphasis-style Emphasis style [Expected: asterisk; Actual: underscore]
packages/pi-continuous-learning/README.md:211:43 MD049/emphasis-style Emphasis style [Expected: asterisk; Actual: underscore]
packages/pi-continuous-learning/README.md:211:46 MD049/emphasis-style Emphasis style [Expected: asterisk; Actual: underscore]
packages/pi-continuous-learning/README.md:211:64 MD049/emphasis-style Emphasis style [Expected: asterisk; Actual: underscore]
packages/pi-continuous-learning/README.md:211:70 MD049/emphasis-style Emphasis style [Expected: asterisk; Actual: underscore]
packages/pi-continuous-learning/README.md:211:106 MD049/emphasis-style Emphasis style [Expected: asterisk; Actual: underscore]

See detailed reports in MegaLinter artifacts
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@MattDevy
MattDevy merged commit 7312fa5 into main Jul 17, 2026
6 checks passed
@MattDevy
MattDevy deleted the fix/pi-cl-custom-providers branch July 17, 2026 14:30
@MattDevy MattDevy mentioned this pull request Jul 17, 2026
MattDevy added a commit that referenced this pull request Jul 17, 2026
🤖 I have created a release *beep* *boop*
---


<details><summary>pi-continuous-learning: 0.14.3</summary>

##
[0.14.3](pi-continuous-learning-v0.14.2...pi-continuous-learning-v0.14.3)
(2026-07-17)


### Bug Fixes

* **pi-continuous-learning:** Support custom analyzer providers
([#154](#154))
([7312fa5](7312fa5)),
closes [#134](#134)
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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.

pi-cl-analyze does not support custom providers from models.json

1 participant