Skip to content

feat(ide): completion#6

Merged
roife merged 68 commits into
pascal-lab:masterfrom
hongjr03:feature/ide-completion-core
Apr 14, 2026
Merged

feat(ide): completion#6
roife merged 68 commits into
pascal-lab:masterfrom
hongjr03:feature/ide-completion-core

Conversation

@hongjr03
Copy link
Copy Markdown
Member

@hongjr03 hongjr03 commented Nov 4, 2025

No description provided.

@hongjr03 hongjr03 marked this pull request as draft November 4, 2025 12:30
@hongjr03 hongjr03 deleted the branch pascal-lab:master November 29, 2025 11:11
@hongjr03 hongjr03 closed this Nov 29, 2025
@hongjr03 hongjr03 reopened this Nov 30, 2025
@hongjr03 hongjr03 force-pushed the feature/ide-completion-core branch from e41cde7 to b8c6cbf Compare January 13, 2026 09:30
@hongjr03 hongjr03 changed the base branch from completion to master January 13, 2026 09:36
@hongjr03 hongjr03 changed the title feat(ide): implement core completion logic feat(ide): completion Jan 13, 2026
@hongjr03 hongjr03 marked this pull request as ready for review January 13, 2026 09:52
@hongjr03 hongjr03 force-pushed the feature/ide-completion-core branch from 42fa44a to f79b736 Compare January 17, 2026 05:08
@hongjr03 hongjr03 force-pushed the feature/ide-completion-core branch from f79b736 to e0857d4 Compare January 17, 2026 06:49
…clarator names

- Add `in_decl_name` field to CompletionContext to detect when cursor is
  at a new symbol name position (e.g., after type in port declaration)
- Fix line comment detection when comment appears before preprocessor
  directives like `timescale - now checks nested trivia inside directive
- Change trivia_at_offset to return TriviaKind instead of SyntaxTrivia
  to avoid lifetime issues with nested trivia
- Add comprehensive tests for line comment detection in various positions
…letion

- Distinguish between value names and subroutine names in completion
- Generate snippet with placeholder for subroutine calls: func(${1:args})
@hongjr03 hongjr03 force-pushed the feature/ide-completion-core branch from 6f1db83 to 6be92e9 Compare January 17, 2026 16:13
@roife roife merged commit 4d8fd50 into pascal-lab:master Apr 14, 2026
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.

2 participants