@cluesmith/codev-core 3.1.9 → 3.2.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/architect-name.d.ts +44 -0
- package/dist/architect-name.d.ts.map +1 -0
- package/dist/architect-name.js +73 -0
- package/dist/architect-name.js.map +1 -0
- package/dist/review-markers.d.ts +81 -0
- package/dist/review-markers.d.ts.map +1 -0
- package/dist/review-markers.js +97 -0
- package/dist/review-markers.js.map +1 -0
- package/package.json +10 -2
|
@@ -0,0 +1,44 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Architect-name utilities shared across packages (Spec 755 / 786).
|
|
3
|
+
*
|
|
4
|
+
* Lives in codev-core so the VS Code extension and the agent-farm server can
|
|
5
|
+
* both validate architect names with identical semantics — the extension's
|
|
6
|
+
* "Add Architect" InputBox (Issue 841) reuses the same rule Tower enforces
|
|
7
|
+
* server-side, instead of duplicating the regex.
|
|
8
|
+
*
|
|
9
|
+
* These functions are pure — they don't read process state or Tower; the caller
|
|
10
|
+
* is responsible for sourcing the existing-names set. Keeping them pure makes
|
|
11
|
+
* them trivially unit-testable. (`currentArchitectName`, which reads the
|
|
12
|
+
* process env, stays in the agent-farm package.)
|
|
13
|
+
*/
|
|
14
|
+
export declare const ARCHITECT_NAME_PATTERN: RegExp;
|
|
15
|
+
export declare const MAX_ARCHITECT_NAME_LENGTH = 64;
|
|
16
|
+
/** Reserved default for the singleton case. */
|
|
17
|
+
export declare const DEFAULT_ARCHITECT_NAME = "main";
|
|
18
|
+
/**
|
|
19
|
+
* Validate an architect name. Returns `null` if valid, or a human-readable
|
|
20
|
+
* error message otherwise. Callers should treat a non-null return as "reject
|
|
21
|
+
* with this message" — the text is intentionally operator-facing.
|
|
22
|
+
*
|
|
23
|
+
* Spec 786: the name `main` is reserved for the workspace's default architect
|
|
24
|
+
* and is rejected here. Pre-#786, `main` was rejected only by collision check
|
|
25
|
+
* at the add-architect call site (which depended on a race-free in-memory map);
|
|
26
|
+
* rejecting in the pure validator is more robust.
|
|
27
|
+
*/
|
|
28
|
+
export declare function validateArchitectName(name: string): string | null;
|
|
29
|
+
/**
|
|
30
|
+
* Compute the next auto-numbered architect name, given the set of names
|
|
31
|
+
* already in use. Uses "smallest unused integer ≥ 2" semantics:
|
|
32
|
+
*
|
|
33
|
+
* - {} → 'architect-2'
|
|
34
|
+
* - {'main'} → 'architect-2'
|
|
35
|
+
* - {'main', 'architect-2'} → 'architect-3'
|
|
36
|
+
* - {'main', 'architect-3'} → 'architect-2' (fills the gap)
|
|
37
|
+
* - {'main', 'architect-2', 'architect-3'} → 'architect-4'
|
|
38
|
+
* - {'main', 'sibling'} → 'architect-2' (custom names don't shift numbering)
|
|
39
|
+
*
|
|
40
|
+
* Custom names (anything not matching `architect-<N>` exactly) are ignored
|
|
41
|
+
* by the numbering loop — they're not part of the auto-numbered sequence.
|
|
42
|
+
*/
|
|
43
|
+
export declare function autoNumberArchitectName(existingNames: Iterable<string>): string;
|
|
44
|
+
//# sourceMappingURL=architect-name.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"architect-name.d.ts","sourceRoot":"","sources":["../src/architect-name.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,eAAO,MAAM,sBAAsB,QAAsB,CAAC;AAC1D,eAAO,MAAM,yBAAyB,KAAK,CAAC;AAE5C,+CAA+C;AAC/C,eAAO,MAAM,sBAAsB,SAAS,CAAC;AAE7C;;;;;;;;;GASG;AACH,wBAAgB,qBAAqB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CAcjE;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAgB,uBAAuB,CAAC,aAAa,EAAE,QAAQ,CAAC,MAAM,CAAC,GAAG,MAAM,CAc/E"}
|
|
@@ -0,0 +1,73 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Architect-name utilities shared across packages (Spec 755 / 786).
|
|
3
|
+
*
|
|
4
|
+
* Lives in codev-core so the VS Code extension and the agent-farm server can
|
|
5
|
+
* both validate architect names with identical semantics — the extension's
|
|
6
|
+
* "Add Architect" InputBox (Issue 841) reuses the same rule Tower enforces
|
|
7
|
+
* server-side, instead of duplicating the regex.
|
|
8
|
+
*
|
|
9
|
+
* These functions are pure — they don't read process state or Tower; the caller
|
|
10
|
+
* is responsible for sourcing the existing-names set. Keeping them pure makes
|
|
11
|
+
* them trivially unit-testable. (`currentArchitectName`, which reads the
|
|
12
|
+
* process env, stays in the agent-farm package.)
|
|
13
|
+
*/
|
|
14
|
+
export const ARCHITECT_NAME_PATTERN = /^[a-z][a-z0-9-]*$/;
|
|
15
|
+
export const MAX_ARCHITECT_NAME_LENGTH = 64;
|
|
16
|
+
/** Reserved default for the singleton case. */
|
|
17
|
+
export const DEFAULT_ARCHITECT_NAME = 'main';
|
|
18
|
+
/**
|
|
19
|
+
* Validate an architect name. Returns `null` if valid, or a human-readable
|
|
20
|
+
* error message otherwise. Callers should treat a non-null return as "reject
|
|
21
|
+
* with this message" — the text is intentionally operator-facing.
|
|
22
|
+
*
|
|
23
|
+
* Spec 786: the name `main` is reserved for the workspace's default architect
|
|
24
|
+
* and is rejected here. Pre-#786, `main` was rejected only by collision check
|
|
25
|
+
* at the add-architect call site (which depended on a race-free in-memory map);
|
|
26
|
+
* rejecting in the pure validator is more robust.
|
|
27
|
+
*/
|
|
28
|
+
export function validateArchitectName(name) {
|
|
29
|
+
if (!name) {
|
|
30
|
+
return 'Architect name cannot be empty.';
|
|
31
|
+
}
|
|
32
|
+
if (name === DEFAULT_ARCHITECT_NAME) {
|
|
33
|
+
return `Architect name '${DEFAULT_ARCHITECT_NAME}' is reserved for the workspace's default architect.`;
|
|
34
|
+
}
|
|
35
|
+
if (name.length > MAX_ARCHITECT_NAME_LENGTH) {
|
|
36
|
+
return `Architect name must be at most ${MAX_ARCHITECT_NAME_LENGTH} characters (got ${name.length}).`;
|
|
37
|
+
}
|
|
38
|
+
if (!ARCHITECT_NAME_PATTERN.test(name)) {
|
|
39
|
+
return `Architect name '${name}' is invalid. Names must match [a-z][a-z0-9-]* (lowercase letter, then lowercase letters / digits / dashes).`;
|
|
40
|
+
}
|
|
41
|
+
return null;
|
|
42
|
+
}
|
|
43
|
+
/**
|
|
44
|
+
* Compute the next auto-numbered architect name, given the set of names
|
|
45
|
+
* already in use. Uses "smallest unused integer ≥ 2" semantics:
|
|
46
|
+
*
|
|
47
|
+
* - {} → 'architect-2'
|
|
48
|
+
* - {'main'} → 'architect-2'
|
|
49
|
+
* - {'main', 'architect-2'} → 'architect-3'
|
|
50
|
+
* - {'main', 'architect-3'} → 'architect-2' (fills the gap)
|
|
51
|
+
* - {'main', 'architect-2', 'architect-3'} → 'architect-4'
|
|
52
|
+
* - {'main', 'sibling'} → 'architect-2' (custom names don't shift numbering)
|
|
53
|
+
*
|
|
54
|
+
* Custom names (anything not matching `architect-<N>` exactly) are ignored
|
|
55
|
+
* by the numbering loop — they're not part of the auto-numbered sequence.
|
|
56
|
+
*/
|
|
57
|
+
export function autoNumberArchitectName(existingNames) {
|
|
58
|
+
const usedNumbers = new Set();
|
|
59
|
+
for (const name of existingNames) {
|
|
60
|
+
const match = /^architect-(\d+)$/.exec(name);
|
|
61
|
+
if (match) {
|
|
62
|
+
const n = Number.parseInt(match[1], 10);
|
|
63
|
+
if (Number.isFinite(n) && n >= 2) {
|
|
64
|
+
usedNumbers.add(n);
|
|
65
|
+
}
|
|
66
|
+
}
|
|
67
|
+
}
|
|
68
|
+
let n = 2;
|
|
69
|
+
while (usedNumbers.has(n))
|
|
70
|
+
n++;
|
|
71
|
+
return `architect-${n}`;
|
|
72
|
+
}
|
|
73
|
+
//# sourceMappingURL=architect-name.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"architect-name.js","sourceRoot":"","sources":["../src/architect-name.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,MAAM,CAAC,MAAM,sBAAsB,GAAG,mBAAmB,CAAC;AAC1D,MAAM,CAAC,MAAM,yBAAyB,GAAG,EAAE,CAAC;AAE5C,+CAA+C;AAC/C,MAAM,CAAC,MAAM,sBAAsB,GAAG,MAAM,CAAC;AAE7C;;;;;;;;;GASG;AACH,MAAM,UAAU,qBAAqB,CAAC,IAAY;IAChD,IAAI,CAAC,IAAI,EAAE,CAAC;QACV,OAAO,iCAAiC,CAAC;IAC3C,CAAC;IACD,IAAI,IAAI,KAAK,sBAAsB,EAAE,CAAC;QACpC,OAAO,mBAAmB,sBAAsB,sDAAsD,CAAC;IACzG,CAAC;IACD,IAAI,IAAI,CAAC,MAAM,GAAG,yBAAyB,EAAE,CAAC;QAC5C,OAAO,kCAAkC,yBAAyB,oBAAoB,IAAI,CAAC,MAAM,IAAI,CAAC;IACxG,CAAC;IACD,IAAI,CAAC,sBAAsB,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;QACvC,OAAO,mBAAmB,IAAI,8GAA8G,CAAC;IAC/I,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;;;;;;;;;;;;GAaG;AACH,MAAM,UAAU,uBAAuB,CAAC,aAA+B;IACrE,MAAM,WAAW,GAAG,IAAI,GAAG,EAAU,CAAC;IACtC,KAAK,MAAM,IAAI,IAAI,aAAa,EAAE,CAAC;QACjC,MAAM,KAAK,GAAG,mBAAmB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC7C,IAAI,KAAK,EAAE,CAAC;YACV,MAAM,CAAC,GAAG,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YACxC,IAAI,MAAM,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;gBACjC,WAAW,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;YACrB,CAAC;QACH,CAAC;IACH,CAAC;IACD,IAAI,CAAC,GAAG,CAAC,CAAC;IACV,OAAO,WAAW,CAAC,GAAG,CAAC,CAAC,CAAC;QAAE,CAAC,EAAE,CAAC;IAC/B,OAAO,aAAa,CAAC,EAAE,CAAC;AAC1B,CAAC"}
|
|
@@ -0,0 +1,81 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Review-marker codec — the single, host-agnostic home for the on-disk `REVIEW`
|
|
3
|
+
* comment format and the transforms a rendered review surface needs.
|
|
4
|
+
*
|
|
5
|
+
* Background (#859): `@cluesmith/codev-artifact-canvas` renders Codev artifacts
|
|
6
|
+
* (specs/plans/reviews) and lets a reviewer add inline comments. A comment is
|
|
7
|
+
* serialized into the markdown as a positional HTML comment:
|
|
8
|
+
*
|
|
9
|
+
* <!-- REVIEW(@<author>): <text> -->
|
|
10
|
+
*
|
|
11
|
+
* written on the line **after** the block it annotates. There is no `line=N`
|
|
12
|
+
* attribute. The marker's *position in the file* is the entire line association:
|
|
13
|
+
* a marker annotates the nearest non-marker line above it (so a stack of
|
|
14
|
+
* comments on one block all resolve to that block). This convention
|
|
15
|
+
* matches both the existing VSCode editor Comments-API path
|
|
16
|
+
* (`packages/vscode/src/comments/plan-review.ts`) and the canvas package's own
|
|
17
|
+
* tested adapter contract, so the two authoring surfaces round-trip through the
|
|
18
|
+
* same bytes.
|
|
19
|
+
*
|
|
20
|
+
* This module lives in `core` (not in a host) because the convention is a
|
|
21
|
+
* cross-host contract: the VSCode host consumes it now, and the dashboard /
|
|
22
|
+
* future hosts consume the same functions so they cannot drift. It is pure
|
|
23
|
+
* (no `vscode`, no DOM, no I/O) and therefore safe in both Node and the browser.
|
|
24
|
+
*
|
|
25
|
+
* Hiding markers from the rendered body is NOT done here: the canvas renderer
|
|
26
|
+
* strips full-line HTML comments before parsing (keeping blocks intact and
|
|
27
|
+
* `data-line` accurate), so hosts pass raw document text straight through. This
|
|
28
|
+
* codec is purely the on-disk read/write convention (#1036 is fixed in the
|
|
29
|
+
* renderer, #1042).
|
|
30
|
+
*/
|
|
31
|
+
/**
|
|
32
|
+
* In-memory model of one review marker. Structurally matches
|
|
33
|
+
* `ReviewMarker` from `@cluesmith/codev-artifact-canvas` (kept as a local
|
|
34
|
+
* declaration so `core` does not depend on the React rendering package).
|
|
35
|
+
*/
|
|
36
|
+
export interface ReviewMarker {
|
|
37
|
+
author: string;
|
|
38
|
+
/** 0-based logical line the marker annotates (the line *above* the marker). */
|
|
39
|
+
line: number;
|
|
40
|
+
text: string;
|
|
41
|
+
/** Original on-disk marker text, for lossless round-tripping. */
|
|
42
|
+
raw: string;
|
|
43
|
+
/** Reserved for region anchors (unused in v1). */
|
|
44
|
+
lineRange?: {
|
|
45
|
+
start: number;
|
|
46
|
+
end: number;
|
|
47
|
+
};
|
|
48
|
+
}
|
|
49
|
+
/**
|
|
50
|
+
* Eligible artifact paths — the `+` affordance and the canvas editor are gated
|
|
51
|
+
* to these. Shared so the editor path and the canvas host agree on one rule.
|
|
52
|
+
*/
|
|
53
|
+
export declare const ELIGIBLE_PATH_REGEX: RegExp;
|
|
54
|
+
/** True if `fsPath` (forward-slash normalized or not) is a review-eligible artifact. */
|
|
55
|
+
export declare function isEligibleReviewPath(fsPath: string): boolean;
|
|
56
|
+
/** True if a single line is a review marker. */
|
|
57
|
+
export declare function isReviewMarkerLine(line: string): boolean;
|
|
58
|
+
/**
|
|
59
|
+
* Serialize a marker to its canonical on-disk form. The body is
|
|
60
|
+
* whitespace-normalized to a single line (markers are single-line by
|
|
61
|
+
* convention; the canvas/editor decorations assume one line). `indent` is
|
|
62
|
+
* prepended so a marker inside an indented list item keeps the list's
|
|
63
|
+
* indentation.
|
|
64
|
+
*/
|
|
65
|
+
export declare function serializeReviewMarker(author: string, body: string, indent?: string): string;
|
|
66
|
+
/**
|
|
67
|
+
* The file line a new marker is written to, given the 0-based line it annotates:
|
|
68
|
+
* the line *after* the annotated line. Encoded here so every caller (editor +
|
|
69
|
+
* canvas host) shares one definition of the convention and cannot drift.
|
|
70
|
+
*/
|
|
71
|
+
export declare function markerInsertionLine(annotatedLine: number): number;
|
|
72
|
+
/**
|
|
73
|
+
* Parse all review markers out of `text` into `ReviewMarker`s, mapping each to
|
|
74
|
+
* the 0-based logical line it annotates: the nearest line ABOVE the marker that
|
|
75
|
+
* is not itself a marker. Skipping over a run of stacked markers means several
|
|
76
|
+
* comments on one block all anchor to that block's start (the canvas renders
|
|
77
|
+
* them as a list) instead of each pointing at the marker above it. A marker, or
|
|
78
|
+
* a run of markers, at the very top of the file annotates nothing and is skipped.
|
|
79
|
+
*/
|
|
80
|
+
export declare function parseReviewMarkers(text: string): ReviewMarker[];
|
|
81
|
+
//# sourceMappingURL=review-markers.d.ts.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"review-markers.d.ts","sourceRoot":"","sources":["../src/review-markers.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA6BG;AAEH;;;;GAIG;AACH,MAAM,WAAW,YAAY;IAC3B,MAAM,EAAE,MAAM,CAAC;IACf,+EAA+E;IAC/E,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,iEAAiE;IACjE,GAAG,EAAE,MAAM,CAAC;IACZ,kDAAkD;IAClD,SAAS,CAAC,EAAE;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,GAAG,EAAE,MAAM,CAAA;KAAE,CAAC;CAC5C;AAED;;;GAGG;AACH,eAAO,MAAM,mBAAmB,QAAqC,CAAC;AAUtE,wFAAwF;AACxF,wBAAgB,oBAAoB,CAAC,MAAM,EAAE,MAAM,GAAG,OAAO,CAE5D;AAED,gDAAgD;AAChD,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,GAAG,OAAO,CAExD;AAED;;;;;;GAMG;AACH,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,SAAK,GAAG,MAAM,CAGvF;AAED;;;;GAIG;AACH,wBAAgB,mBAAmB,CAAC,aAAa,EAAE,MAAM,GAAG,MAAM,CAEjE;AAED;;;;;;;GAOG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,GAAG,YAAY,EAAE,CAY/D"}
|
|
@@ -0,0 +1,97 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Review-marker codec — the single, host-agnostic home for the on-disk `REVIEW`
|
|
3
|
+
* comment format and the transforms a rendered review surface needs.
|
|
4
|
+
*
|
|
5
|
+
* Background (#859): `@cluesmith/codev-artifact-canvas` renders Codev artifacts
|
|
6
|
+
* (specs/plans/reviews) and lets a reviewer add inline comments. A comment is
|
|
7
|
+
* serialized into the markdown as a positional HTML comment:
|
|
8
|
+
*
|
|
9
|
+
* <!-- REVIEW(@<author>): <text> -->
|
|
10
|
+
*
|
|
11
|
+
* written on the line **after** the block it annotates. There is no `line=N`
|
|
12
|
+
* attribute. The marker's *position in the file* is the entire line association:
|
|
13
|
+
* a marker annotates the nearest non-marker line above it (so a stack of
|
|
14
|
+
* comments on one block all resolve to that block). This convention
|
|
15
|
+
* matches both the existing VSCode editor Comments-API path
|
|
16
|
+
* (`packages/vscode/src/comments/plan-review.ts`) and the canvas package's own
|
|
17
|
+
* tested adapter contract, so the two authoring surfaces round-trip through the
|
|
18
|
+
* same bytes.
|
|
19
|
+
*
|
|
20
|
+
* This module lives in `core` (not in a host) because the convention is a
|
|
21
|
+
* cross-host contract: the VSCode host consumes it now, and the dashboard /
|
|
22
|
+
* future hosts consume the same functions so they cannot drift. It is pure
|
|
23
|
+
* (no `vscode`, no DOM, no I/O) and therefore safe in both Node and the browser.
|
|
24
|
+
*
|
|
25
|
+
* Hiding markers from the rendered body is NOT done here: the canvas renderer
|
|
26
|
+
* strips full-line HTML comments before parsing (keeping blocks intact and
|
|
27
|
+
* `data-line` accurate), so hosts pass raw document text straight through. This
|
|
28
|
+
* codec is purely the on-disk read/write convention (#1036 is fixed in the
|
|
29
|
+
* renderer, #1042).
|
|
30
|
+
*/
|
|
31
|
+
/**
|
|
32
|
+
* Eligible artifact paths — the `+` affordance and the canvas editor are gated
|
|
33
|
+
* to these. Shared so the editor path and the canvas host agree on one rule.
|
|
34
|
+
*/
|
|
35
|
+
export const ELIGIBLE_PATH_REGEX = /\/codev\/(plans|specs|reviews)\//;
|
|
36
|
+
/**
|
|
37
|
+
* Per-line marker matcher. Capture groups: [1] indent, [2] author, [3] body.
|
|
38
|
+
* A marker occupies its own line by convention; the body is single-line
|
|
39
|
+
* (whitespace-normalized at write time). Tolerant of surrounding whitespace to
|
|
40
|
+
* stay compatible with markers authored by the editor path and by hand.
|
|
41
|
+
*/
|
|
42
|
+
const REVIEW_MARKER_RE = /^(\s*)<!--\s*REVIEW\s*\(@([^)]+)\)\s*:\s*([\s\S]*?)\s*-->\s*$/;
|
|
43
|
+
/** True if `fsPath` (forward-slash normalized or not) is a review-eligible artifact. */
|
|
44
|
+
export function isEligibleReviewPath(fsPath) {
|
|
45
|
+
return ELIGIBLE_PATH_REGEX.test(fsPath.replace(/\\/g, '/'));
|
|
46
|
+
}
|
|
47
|
+
/** True if a single line is a review marker. */
|
|
48
|
+
export function isReviewMarkerLine(line) {
|
|
49
|
+
return REVIEW_MARKER_RE.test(line);
|
|
50
|
+
}
|
|
51
|
+
/**
|
|
52
|
+
* Serialize a marker to its canonical on-disk form. The body is
|
|
53
|
+
* whitespace-normalized to a single line (markers are single-line by
|
|
54
|
+
* convention; the canvas/editor decorations assume one line). `indent` is
|
|
55
|
+
* prepended so a marker inside an indented list item keeps the list's
|
|
56
|
+
* indentation.
|
|
57
|
+
*/
|
|
58
|
+
export function serializeReviewMarker(author, body, indent = '') {
|
|
59
|
+
const normalized = body.replace(/\s+/g, ' ').trim();
|
|
60
|
+
return `${indent}<!-- REVIEW(@${author}): ${normalized} -->`;
|
|
61
|
+
}
|
|
62
|
+
/**
|
|
63
|
+
* The file line a new marker is written to, given the 0-based line it annotates:
|
|
64
|
+
* the line *after* the annotated line. Encoded here so every caller (editor +
|
|
65
|
+
* canvas host) shares one definition of the convention and cannot drift.
|
|
66
|
+
*/
|
|
67
|
+
export function markerInsertionLine(annotatedLine) {
|
|
68
|
+
return annotatedLine + 1;
|
|
69
|
+
}
|
|
70
|
+
/**
|
|
71
|
+
* Parse all review markers out of `text` into `ReviewMarker`s, mapping each to
|
|
72
|
+
* the 0-based logical line it annotates: the nearest line ABOVE the marker that
|
|
73
|
+
* is not itself a marker. Skipping over a run of stacked markers means several
|
|
74
|
+
* comments on one block all anchor to that block's start (the canvas renders
|
|
75
|
+
* them as a list) instead of each pointing at the marker above it. A marker, or
|
|
76
|
+
* a run of markers, at the very top of the file annotates nothing and is skipped.
|
|
77
|
+
*/
|
|
78
|
+
export function parseReviewMarkers(text) {
|
|
79
|
+
const out = [];
|
|
80
|
+
const lines = text.split('\n');
|
|
81
|
+
for (let i = 0; i < lines.length; i++) {
|
|
82
|
+
const m = REVIEW_MARKER_RE.exec(lines[i]);
|
|
83
|
+
if (!m) {
|
|
84
|
+
continue;
|
|
85
|
+
}
|
|
86
|
+
let anchor = i - 1;
|
|
87
|
+
while (anchor >= 0 && isReviewMarkerLine(lines[anchor])) {
|
|
88
|
+
anchor--;
|
|
89
|
+
}
|
|
90
|
+
if (anchor < 0) {
|
|
91
|
+
continue;
|
|
92
|
+
}
|
|
93
|
+
out.push({ author: m[2], line: anchor, text: m[3], raw: lines[i].trim() });
|
|
94
|
+
}
|
|
95
|
+
return out;
|
|
96
|
+
}
|
|
97
|
+
//# sourceMappingURL=review-markers.js.map
|
|
@@ -0,0 +1 @@
|
|
|
1
|
+
{"version":3,"file":"review-markers.js","sourceRoot":"","sources":["../src/review-markers.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA6BG;AAkBH;;;GAGG;AACH,MAAM,CAAC,MAAM,mBAAmB,GAAG,kCAAkC,CAAC;AAEtE;;;;;GAKG;AACH,MAAM,gBAAgB,GAAG,+DAA+D,CAAC;AAEzF,wFAAwF;AACxF,MAAM,UAAU,oBAAoB,CAAC,MAAc;IACjD,OAAO,mBAAmB,CAAC,IAAI,CAAC,MAAM,CAAC,OAAO,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC,CAAC;AAC9D,CAAC;AAED,gDAAgD;AAChD,MAAM,UAAU,kBAAkB,CAAC,IAAY;IAC7C,OAAO,gBAAgB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACrC,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,qBAAqB,CAAC,MAAc,EAAE,IAAY,EAAE,MAAM,GAAG,EAAE;IAC7E,MAAM,UAAU,GAAG,IAAI,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC;IACpD,OAAO,GAAG,MAAM,gBAAgB,MAAM,MAAM,UAAU,MAAM,CAAC;AAC/D,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,mBAAmB,CAAC,aAAqB;IACvD,OAAO,aAAa,GAAG,CAAC,CAAC;AAC3B,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,kBAAkB,CAAC,IAAY;IAC7C,MAAM,GAAG,GAAmB,EAAE,CAAC;IAC/B,MAAM,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC/B,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,KAAK,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACtC,MAAM,CAAC,GAAG,gBAAgB,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QAC1C,IAAI,CAAC,CAAC,EAAE,CAAC;YAAC,SAAS;QAAC,CAAC;QACrB,IAAI,MAAM,GAAG,CAAC,GAAG,CAAC,CAAC;QACnB,OAAO,MAAM,IAAI,CAAC,IAAI,kBAAkB,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,EAAE,CAAC;YAAC,MAAM,EAAE,CAAC;QAAC,CAAC;QACtE,IAAI,MAAM,GAAG,CAAC,EAAE,CAAC;YAAC,SAAS;QAAC,CAAC;QAC7B,GAAG,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,GAAG,EAAE,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;IAC7E,CAAC;IACD,OAAO,GAAG,CAAC;AACb,CAAC"}
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@cluesmith/codev-core",
|
|
3
|
-
"version": "3.1
|
|
3
|
+
"version": "3.2.1",
|
|
4
4
|
"description": "Core runtime utilities for Codev — Tower client, auth, workspace helpers, escape buffer",
|
|
5
5
|
"type": "module",
|
|
6
6
|
"exports": {
|
|
@@ -32,6 +32,10 @@
|
|
|
32
32
|
"types": "./dist/agent-names.d.ts",
|
|
33
33
|
"default": "./dist/agent-names.js"
|
|
34
34
|
},
|
|
35
|
+
"./architect-name": {
|
|
36
|
+
"types": "./dist/architect-name.d.ts",
|
|
37
|
+
"default": "./dist/architect-name.js"
|
|
38
|
+
},
|
|
35
39
|
"./builder-helpers": {
|
|
36
40
|
"types": "./dist/builder-helpers.d.ts",
|
|
37
41
|
"default": "./dist/builder-helpers.js"
|
|
@@ -43,6 +47,10 @@
|
|
|
43
47
|
"./phase-grouping": {
|
|
44
48
|
"types": "./dist/phase-grouping.d.ts",
|
|
45
49
|
"default": "./dist/phase-grouping.js"
|
|
50
|
+
},
|
|
51
|
+
"./review-markers": {
|
|
52
|
+
"types": "./dist/review-markers.d.ts",
|
|
53
|
+
"default": "./dist/review-markers.js"
|
|
46
54
|
}
|
|
47
55
|
},
|
|
48
56
|
"files": [
|
|
@@ -52,7 +60,7 @@
|
|
|
52
60
|
"@types/node": "22.x",
|
|
53
61
|
"typescript": "^5.7.0",
|
|
54
62
|
"vitest": "^4.0.15",
|
|
55
|
-
"@cluesmith/codev-types": "3.1
|
|
63
|
+
"@cluesmith/codev-types": "3.2.1"
|
|
56
64
|
},
|
|
57
65
|
"license": "Apache-2.0",
|
|
58
66
|
"scripts": {
|