upstream/prometheus
1
0
Fork 0
mirror of https://github.com/prometheus/prometheus.git synced 2026-06-03 21:52:13 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 14

fix: autocomplete suggestions for using cursor position

Browse source 
Signed-off-by: ADITYA TIWARI <adityatiwari342005@gmail.com>
This commit is contained in:
ADITYA TIWARI 2025-11-24 09:05:48 +00:00
parent d0d2699dc5
commit 02f405692e
2 changed files with 13 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
web/ui/module/codemirror-promql/src/complete/hybrid.test.ts
View file

@ -299,6 +299,12 @@ describe('analyzeCompletion test', () => {
pos: 33, // cursor is between the bracket after the comma
expectedContext: [{ kind: ContextKind.LabelName, metricName: 'metric_name' }],
},
{
title: 'no label suggestions after closing matcher',
expr: 'up{job="prometheus"}',
pos: 20, // cursor is right after the closing curly bracket
expectedContext: [],
},
{
title: 'continue autocomplete labelName that defined a metric',
expr: '{myL}',

8
web/ui/module/codemirror-promql/src/complete/hybrid.ts
View file

@ -400,12 +400,18 @@ export function analyzeCompletion(state: EditorState, node: SyntaxNode, pos: num
// so we have or to autocomplete any kind of labelName or to autocomplete only the labelName associated to the metric
result.push({ kind: ContextKind.LabelName, metricName: getMetricNameInGroupBy(node, state) });
break;
case LabelMatchers:
case LabelMatchers: {
if (pos >= node.to) {
// Cursor is outside of the label matcher block (e.g. right after `}`),
// so don't offer label-related completions anymore.
break;
}
// In that case we are in the given situation:
// metric_name{} or {}
// so we have or to autocomplete any kind of labelName or to autocomplete only the labelName associated to the metric
result.push({ kind: ContextKind.LabelName, metricName: getMetricNameInVectorSelector(node, state) });
break;
}
case LabelName:
if (node.parent?.type.id === GroupingLabels) {
// In this case we are in the given situation: