Skip to content

Commit 860084c

Browse files
committed
fixing build
1 parent b550db6 commit 860084c

2 files changed

Lines changed: 10 additions & 14 deletions

File tree

src/platform/plugins/shared/workflows_management/public/widgets/workflow_yaml_editor/lib/snippets/insert_step_snippet.ts

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -84,11 +84,14 @@ function findStepWithMaxEndOffsetAtOrBefore(
8484
let bestEndOffset = -1;
8585
for (const node of nodes) {
8686
const range = getMonacoRangeFromYamlNode(model, node);
87-
if (!range) continue;
88-
const endOffset = model.getOffsetAt(new monaco.Position(range.endLineNumber, range.endColumn));
89-
if (endOffset <= cursorOffset && endOffset > bestEndOffset) {
90-
bestEndOffset = endOffset;
91-
best = node;
87+
if (range) {
88+
const endOffset = model.getOffsetAt(
89+
new monaco.Position(range.endLineNumber, range.endColumn)
90+
);
91+
if (endOffset <= cursorOffset && endOffset > bestEndOffset) {
92+
bestEndOffset = endOffset;
93+
best = node;
94+
}
9295
}
9396
}
9497
return best;

src/platform/plugins/shared/workflows_management/public/widgets/workflow_yaml_editor/lib/snippets/snippet_insertion_utils.ts

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -213,15 +213,8 @@ export function getInsertRangeAndTextForTriggers(
213213
const lineAfterNext = nextLineNumber + 1;
214214
const lineAfterNextContent = getLineContent(model, lineAfterNext);
215215
if (lineAfterNextContent === null) {
216-
const lastLineNumber = model.getLineCount();
217-
const lastLineEndColumn = model.getLineMaxColumn(lastLineNumber);
218-
const range = new monaco.Range(
219-
lastLineNumber,
220-
lastLineEndColumn,
221-
lastLineNumber,
222-
lastLineEndColumn
223-
);
224-
return { range, text: `\n${insertText}` };
216+
const range = new monaco.Range(lineAfterNext, 1, lineAfterNext, 1);
217+
return { range, text: insertText };
225218
}
226219
if (lineAfterNextContent.trim() === '') {
227220
const range = new monaco.Range(lineAfterNext, 1, lineAfterNext, 1);

0 commit comments

Comments
 (0)