Skip to content

Conversation

@roomote
Copy link

@roomote roomote bot commented Oct 10, 2025

Summary

This PR attempts to address Issue #8596 by adding support for .al (AL Language) file extensions used in Microsoft Dynamics 365 Business Central development.

Changes

  • Added "al" to the extensions array in src/services/tree-sitter/index.ts
  • Added ".al" to the fallbackExtensions array in src/services/code-index/shared/supported-extensions.ts with appropriate comment

Implementation Details

The .al files will be:

  • Recognized as supported file extensions
  • Included in codebase scanning and indexing
  • Processed using fallback chunking (length-based) instead of tree-sitter parsing since no dedicated WASM parser exists
  • Searchable through codebase retrieval features
  • Available in context for AI assistance

Testing

  • Verified code-index tests pass successfully
  • No regressions introduced
  • Changes follow existing patterns for languages without dedicated parsers (e.g., Visual Basic .NET, Scala, Swift)

Fixes #8596

Feedback and guidance are welcome!


Important

Adds support for .al files with fallback chunking in index.ts and supported-extensions.ts.

  • Behavior:
    • Adds support for .al (AL Language) files in index.ts and supported-extensions.ts.
    • .al files are indexed and processed using fallback chunking due to lack of a dedicated parser.
  • Testing:
    • Verified code-index tests pass.
    • No regressions introduced.
    • Consistent with handling of other languages without dedicated parsers.

This description was created by Ellipsis for 440412d. You can customize this summary. It will automatically update as commits are pushed.

…chunking

- Added "al" to the extensions array in tree-sitter index
- Added ".al" to fallbackExtensions for length-based chunking
- Enables AL Language (Dynamics 365 Business Central) file support

Fixes #8596
@roomote roomote bot requested review from cte, jr and mrubens as code owners October 10, 2025 12:56
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Oct 10, 2025
@dosubot dosubot bot added the enhancement New feature or request label Oct 10, 2025
Copy link
Author

@roomote roomote bot left a comment

Choose a reason for hiding this comment

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

No new issues found - the implementation correctly follows the established pattern for adding fallback extensions. The changes are minimal, well-documented, and consistent with existing code.

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Oct 10, 2025
@yavpungggi yavpungggi mentioned this pull request Oct 10, 2025
6 tasks
@yavpungggi
Copy link

yavpungggi commented Oct 10, 2025

Opened new PR #8606 with the branch from roomote, but added test parser.vb.spec

@hannesrudolph can be closed

@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Prelim Review] in Roo Code Roadmap Oct 28, 2025
@daniel-lxs daniel-lxs closed this Oct 28, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Oct 28, 2025
@github-project-automation github-project-automation bot moved this from PR [Needs Prelim Review] to Done in Roo Code Roadmap Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[ENHANCEMENT] Add support for .al (AL Language) file extension with fallback chunking

5 participants