Replace the integration_openai commands that fail as written - #10
Merged
Merged
Conversation
The admin /models GET answers 412 with basic auth, so the model refresh now runs the RefreshModels job through occ. composer runs as the host uid, since www-data cannot create vendor/ in a host-owned checkout. app:install --force replaces the install-then-enable dance on master, -it is dropped from the redis flush, and the no-scripts install says which providers it breaks. Signed-off-by: Oleksander Piskun <oleksandr2088@icloud.com>
oleksandr-nc
force-pushed
the
ai-stack/verified-commands
branch
from
September 14, 2026 12:48
0bb8870 to
c96df50
Compare
Signed-off-by: Oleksander Piskun <oleksandr2088@icloud.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #6, re-running its commands on master with
integration_openai5.0.0 against an OpenAI-compatible endpoint. The model-cache finding in #6 is right; two of the commands around it are not:GET /index.php/apps/integration_openai/modelswith basic auth answers412 CSRF check failed(the route has noNoCSRFRequired), so the jq check could never pass. The refresh now runs theRefreshModelsjob viaocc background-job:execute --force-executeand checksconfig:app:get integration_openai models. Verified: the stored list fills and, after the Redis flush, themodelenum oncore:text2textlists the endpoint's models.composer installaswww-datafails in the host-owned clone the no-chown flow produces ("vendor could not be created"). As the host uid withCOMPOSER_HOME=/tmp/composerit works, same pattern as the Node step.Smaller: on master
app:installitself prints "not compatible" (the package is still downloaded), soapp:install --forcereplaces the install-then-enable sequence;-itis dropped from theredis-cli flushalllines because it fails without a TTY; the--no-scriptsinstall now says which providers it breaks (nothing exists underOCA\OpenAi\Vendorwithout the php-scoper hook); and "already required by the jq check" is corrected, the task type is listed even with an empty model list.