Skip to content

Conversation

@kddejong
Copy link
Collaborator

  • Add quote detection and handling in CompletionUtils
  • Add scalar node type detection for better syntax tree parsing
  • Simplify intrinsic function completion providers by using context parameter
  • Improve JSON parsing with incremental parsing support
  • Add comprehensive tests for quote handling in completions

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@kddejong kddejong requested a review from a team as a code owner September 30, 2025 14:45
@kddejong kddejong force-pushed the fix/autocomplete/yamlstrings branch from aedfd85 to 4448691 Compare September 30, 2025 18:04
Copy link
Contributor

@Zee2413 Zee2413 left a comment

Choose a reason for hiding this comment

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

Please rebase before merge.

@kddejong kddejong force-pushed the fix/autocomplete/yamlstrings branch 2 times, most recently from 738ed02 to c930824 Compare September 30, 2025 22:13
@kddejong kddejong force-pushed the fix/autocomplete/yamlstrings branch from c930824 to a4a3733 Compare October 2, 2025 15:16
…tion

- Add quote detection and handling in CompletionUtils
- Add scalar node type detection for better syntax tree parsing
- Simplify intrinsic function completion providers by using context parameter
- Improve JSON parsing with incremental parsing support
- Add comprehensive tests for quote handling in completions
@kddejong kddejong force-pushed the fix/autocomplete/yamlstrings branch from a4a3733 to 589c63b Compare October 2, 2025 15:20
@github-actions
Copy link

github-actions bot commented Oct 2, 2025

Package Line Rate Branch Rate Complexity Health
ai 22% 80% 0
ai.llm 53% 80% 0
app 0% 0% 0
artifacts 100% 100% 0
auth 87% 92% 0
autocomplete 93% 91% 0
context 95% 95% 0
context.semantic 98% 97% 0
context.semantic.parameter 100% 95% 0
context.syntaxtree 89% 90% 0
context.syntaxtree.utils 86% 91% 0
datastore 77% 97% 0
definition 100% 100% 0
document 93% 96% 0
documentSymbol 93% 81% 0
handlers 74% 87% 0
hover 85% 86% 0
protocol 96% 100% 0
resourceState 75% 79% 0
schema 88% 87% 0
schema.transformers 83% 84% 0
server 97% 96% 0
services 66% 89% 0
services.cfnLint 72% 91% 0
services.guard 95% 79% 0
services.guard.assets 81% 77% 0
settings 97% 87% 0
stacks 100% 100% 0
telemetry 58% 78% 0
templates 89% 84% 0
utils 91% 92% 0
Summary 86% (16721 / 19433) 89% (3445 / 3871) 0

@kddejong kddejong merged commit 21789f4 into main Oct 2, 2025
8 checks passed
@kddejong kddejong deleted the fix/autocomplete/yamlstrings branch October 2, 2025 16:27
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.

3 participants