Skip to content

docs: update local model and platform all guidance#17

Merged
ftchvs merged 1 commit into
mainfrom
docs/local-model-platform-all-guidance
May 10, 2026
Merged

docs: update local model and platform all guidance#17
ftchvs merged 1 commit into
mainfrom
docs/local-model-platform-all-guidance

Conversation

@ftchvs
Copy link
Copy Markdown
Owner

@ftchvs ftchvs commented May 10, 2026

Summary

  • Document platform: "all" as a broad cross-platform preflight mode without implying platform parity.
  • Update README/API/local-model guidance for the PR fix: stabilize local model review flow #16 local model UX/runtime behavior: dropdown model discovery, embedding-model filtering, timeout recovery, bounded Ollama generation, think: false, and fenced JSON parsing.
  • Replace stale local model recommendation language with current runtime-compatibility caveats and keep deterministic rules as the trusted baseline.

Validation

  • make test → 196 passed
  • make eval → 58 examples, 1.000 decision accuracy, no decision/policy/category FP/FN notes
  • make benchmark → 213 examples, 1.000 decision accuracy, no decision/policy/category FP/FN notes
  • make pr-preflight → OK

Notes

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1b7aa8e0e7

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread README.md
Comment on lines +492 to +494
If the model endpoint is unavailable or a browser request times out, AdLint
still returns rule-based findings when possible and marks the model status as
`unavailable`.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Remove incorrect timeout fallback claim

The new README text says browser timeouts still return rule-based findings with model.status = unavailable, but the /ui flow aborts the request client-side on timeout (fetchWithTimeout) and then renders an error state without any analysis payload. In the timeout scenario, users do not receive rule findings, so this guidance is misleading for anyone relying on timeout recovery behavior in the web UI.

Useful? React with 👍 / 👎.

@ftchvs ftchvs merged commit ff10d4e into main May 10, 2026
2 checks passed
@ftchvs ftchvs deleted the docs/local-model-platform-all-guidance branch May 13, 2026 14:12
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.

1 participant