feat: include Salesforce Apex files in default patterns#199
Conversation
|
Hi @roian6, Thanks for creating the PR.
Thanks! |
|
Thanks for taking a look — that concern makes sense. I agree My intent was to address #175 from the first-run Apex project discovery angle, but the per-project include_patterns:
- "**/*.cls"
- "**/*.trigger"
language_overrides:
- ext: cls
lang: java
- ext: trigger
lang: javaGiven your point about avoiding broader defaults, I’ll close this PR rather than push the default pattern change further. If you’d like, I can open a separate docs-only PR later with a small Apex settings example instead. |
Summary
Salesforce Apex files are not indexed by default because
.clsclass files and.triggertrigger files are missing fromDEFAULT_INCLUDED_PATTERNS.This keeps the change scoped to default file matching and docs.
detect_code_language()already recognizes.clsand.triggerfiles asapex.Changes
**/*.clsand**/*.triggerto the default include patterns.Closes #175
Verification
uv run pytest tests/test_settings.py::test_default_project_settings -quv run pytest tests/test_settings.py -quv run ruff check src/cocoindex_code/settings.py tests/test_settings.pyuv run mypy src/cocoindex_code/settings.py tests/test_settings.pygit diff --checkuv run prek run --all-files— non-pytest hooks passed (ruff,ruff-format,uv-lock,mypy, and basic file checks); the pytest hook is blocked in this local environment bysqlite_vec/vec0.so: wrong ELF class: ELFCLASS32and a local HuggingFace cache permission issue.