Update docs and add skills.md files#184
Open
dilithjay wants to merge 8 commits into
Open
Conversation
There was a problem hiding this comment.
Pull request overview
This PR expands Lexoid’s documentation to better describe provider setup, CLI usage, and Python API return shapes, and adds SKILL.md guides for the CLI and Python integration.
Changes:
- Add SKILL.md guides for
lexoidCLI usage and the Python API. - Update Sphinx docs to include a new CLI page and refresh API/installation/provider details.
- Adjust docs deployment workflow to only deploy from
main.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| skills/lexoid-python/SKILL.md | New Python skill guide with setup, API entry points, and recipes. |
| skills/lexoid-cli/SKILL.md | New CLI skill guide with commands, flags, and common usage patterns. |
| docs/installation.rst | Clarifies pip install behavior, provider env vars, and optional dependencies (Playwright/LibreOffice/Ollama). |
| docs/index.rst | Updates overview text, feature list, providers list, and adds cli to the toctree. |
| docs/cli.rst | New CLI reference page documenting subcommands and common flags. |
| docs/benchmark.rst | Fixes numbering and expands benchmark configuration descriptions. |
| docs/api.rst | Expands API reference (parser types, kwargs, return keys, examples) and adds missing entry points. |
| .github/workflows/deploy_docs.yml | Ensures the deploy job only runs on refs/heads/main. |
Comments suppressed due to low confidence (3)
docs/api.rst:85
pdf_pathis currently added wheneveras_pdf=True(even whensave_dirisn’t provided, it will point to the temporary converted PDF). The docs say it’s only set whensave_diris specified, which doesn’t matchlexoid.api.parsebehavior.
* ``pdf_path``: Path to the intermediate PDF generated when ``as_pdf=True`` and ``save_dir`` is specified.
skills/lexoid-python/SKILL.md:67
token_costis described as present wheneverapi_cost_mappingis supplied, butparse()only sets it if the mapping contains an entry for the selected model. Consider noting thattoken_costmay be absent (or zero) when the model isn’t in the provided mapping.
"token_usage": {"input": int, "output": int, "total": int, "llm_page_count": int},
"token_cost": {...}, # only when api_cost_mapping is supplied
"parsers_used": [str, ...], # which parser ran per chunk
skills/lexoid-python/SKILL.md:69
- This says
pdf_pathis only present whenas_pdf=Trueandsave_diris set, but the implementation addspdf_pathwheneveras_pdf=True(it may point to a temp file ifsave_dirisn’t provided). Please update this return-shape note to match actual behavior.
"pdf_path": str, # only when as_pdf=True and save_dir is set
}
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Contributor
|
For example, query: For the given document, it defaults to, Should we not consider setting the default config to? |
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
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.
skills/lexoid-cliandskills/lexoid-pythonSKILL.md files for the lexoid CLI and the Python API, respectively.