@open-agent-toolkit/control-plane 0.1.65 → 0.1.69

package/dist/project.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"project.d.ts","sourceRoot":"","sources":["../src/project.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EACV,YAAY,EACZ,cAAc,EAGf,MAAM,SAAS,CAAC;AAEjB,wBAAsB,eAAe,CACnC,WAAW,EAAE,MAAM,GAClB,OAAO,CAAC,YAAY,CAAC,CA8DvB;AAED,wBAAsB,YAAY,CAChC,YAAY,EAAE,MAAM,GACnB,OAAO,CAAC,cAAc,EAAE,CAAC,CAmF3B"}
+{"version":3,"file":"project.d.ts","sourceRoot":"","sources":["../src/project.ts"],"names":[],"mappings":"AAYA,OAAO,KAAK,EACV,YAAY,EACZ,cAAc,EAGf,MAAM,SAAS,CAAC;AAEjB,wBAAsB,eAAe,CACnC,WAAW,EAAE,MAAM,GAClB,OAAO,CAAC,YAAY,CAAC,CA6DvB;AAED,wBAAsB,YAAY,CAChC,YAAY,EAAE,MAAM,GACnB,OAAO,CAAC,cAAc,EAAE,CAAC,CAkF3B"}

package/dist/project.js

@@ -3,7 +3,7 @@ import { basename, dirname, isAbsolute, join, relative } from 'node:path';
 import { recommendSkill } from './recommender/router.js';
 import { scanArtifacts } from './state/artifacts.js';
 import { parseStateFrontmatter } from './state/parser.js';
-import { parseReviewTable, scanUnprocessedReviews } from './state/reviews.js';
+import { parseReviewArtifactIdentity, parseReviewTable, scanUnprocessedReviews, } from './state/reviews.js';
 import { parseTaskProgress } from './state/tasks.js';
 export async function getProjectState(projectPath) {
     const displayPath = await resolveProjectDisplayPath(projectPath);
@@ -15,8 +15,9 @@ export async function getProjectState(projectPath) {
         scanUnprocessedReviews(projectPath),
     ]);
     const parsedState = parseStateFrontmatter(stateContent);
-    const activeReviewArtifacts = toActiveReviewArtifacts(reviewFiles, projectPath);
-    const reviews = mergeReviews(parseReviewTable(planContent), activeReviewArtifacts);
+    const tableReviews = parseReviewTable(planContent);
+    const activeReviewArtifacts = await toActiveReviewArtifacts(reviewFiles, projectPath, tableReviews);
+    const reviews = mergeReviews(tableReviews, activeReviewArtifacts);
     const progress = parseTaskProgress(planContent, implementationContent);
     const projectStateWithoutRecommendation = {
         name: basename(projectPath),
@@ -71,8 +72,9 @@ export async function listProjects(projectsRoot) {
             scanUnprocessedReviews(projectDir),
         ]);
         const parsedState = parseStateFrontmatter(stateContent);
-        const activeReviewArtifacts = toActiveReviewArtifacts(reviewFiles, projectDir);
-        const reviews = mergeReviews(parseReviewTable(planContent), activeReviewArtifacts);
+        const tableReviews = parseReviewTable(planContent);
+        const activeReviewArtifacts = await toActiveReviewArtifacts(reviewFiles, projectDir, tableReviews);
+        const reviews = mergeReviews(tableReviews, activeReviewArtifacts);
         const progress = parseTaskProgress(planContent, implementationContent);
         const stateWithoutRecommendation = {
             name: basename(projectDir),
@@ -192,10 +194,19 @@ function mergeReviews(tableReviews, activeReviewArtifacts) {
     }
     return merged;
 }
-function toActiveReviewArtifacts(reviewFiles, projectPath) {
-    return reviewFiles.map((reviewFile) => ({
-        path: relative(projectPath, reviewFile).replace(/\\/g, '/'),
-        archived: false,
-        actionable: true,
+async function toActiveReviewArtifacts(reviewFiles, projectPath, tableReviews) {
+    return Promise.all(reviewFiles.map(async (reviewFile) => {
+        const path = relative(projectPath, reviewFile).replace(/\\/g, '/');
+        const identity = parseReviewArtifactIdentity(await readOptionalFile(reviewFile));
+        const actionable = identity !== null &&
+            tableReviews.some((review) => review.scope.toLowerCase() === identity.scope.toLowerCase() &&
+                review.type.toLowerCase() === identity.type.toLowerCase() &&
+                review.artifact === path &&
+                review.status.toLowerCase() === 'received');
+        return {
+            path,
+            archived: false,
+            actionable,
+        };
     }));
 }

package/dist/recommender/router.js

@@ -104,7 +104,9 @@ function getPostImplementationRecommendation(state) {
             reason: 'Unprocessed review feedback exists',
         };
     }
-    const finalReview = state.reviews.find((review) => review.scope === 'final' && review.type === 'code');
+    const finalReview = [...state.reviews]
+        .reverse()
+        .find((review) => review.scope === 'final' && review.type === 'code');
     if (!finalReview || finalReview.status === 'pending') {
         return {
             skill: 'oat-project-review-provide',
@@ -148,7 +150,10 @@ function hasIncompleteRevisionPhase(state) {
     return state.progress.phases.some((phase) => phase.isRevision && phase.completed < phase.total);
 }
 function hasActionableReviewArtifacts(state) {
-    return state.activeReviewArtifacts.some((reviewArtifact) => reviewArtifact.actionable && !reviewArtifact.archived);
+    return state.activeReviewArtifacts.some((reviewArtifact) => reviewArtifact.actionable &&
+        !reviewArtifact.archived &&
+        state.reviews.some((review) => review.artifact === reviewArtifact.path &&
+            review.status.toLowerCase() === 'received'));
 }
 function getCurrentArtifact(state) {
     if (state.phase === 'decomposition') {

package/dist/state/reviews.d.ts

@@ -1,4 +1,8 @@
 import type { ReviewStatus } from '../types.js';
 export declare function parseReviewTable(planContent: string): ReviewStatus[];
 export declare function scanUnprocessedReviews(projectPath: string): Promise<string[]>;
+export declare function parseReviewArtifactIdentity(content: string): {
+    scope: string;
+    type: string;
+} | null;
 //# sourceMappingURL=reviews.d.ts.map

package/dist/state/reviews.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"reviews.d.ts","sourceRoot":"","sources":["../../src/state/reviews.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,UAAU,CAAC;AAI7C,wBAAgB,gBAAgB,CAAC,WAAW,EAAE,MAAM,GAAG,YAAY,EAAE,CAepE;AAED,wBAAsB,sBAAsB,CAC1C,WAAW,EAAE,MAAM,GAClB,OAAO,CAAC,MAAM,EAAE,CAAC,CAgBnB"}
+{"version":3,"file":"reviews.d.ts","sourceRoot":"","sources":["../../src/state/reviews.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,UAAU,CAAC;AAI7C,wBAAgB,gBAAgB,CAAC,WAAW,EAAE,MAAM,GAAG,YAAY,EAAE,CAepE;AAED,wBAAsB,sBAAsB,CAC1C,WAAW,EAAE,MAAM,GAClB,OAAO,CAAC,MAAM,EAAE,CAAC,CAgBnB;AAED,wBAAgB,2BAA2B,CACzC,OAAO,EAAE,MAAM,GACd;IAAE,KAAK,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,MAAM,CAAA;CAAE,GAAG,IAAI,CAYxC"}

package/dist/state/reviews.js

@@ -1,6 +1,7 @@
 import { readdir } from 'node:fs/promises';
 import { join } from 'node:path';
 import { isMissingFileError } from '../shared/utils/errors.js';
+import { parseFrontmatterRecord } from '../shared/utils/frontmatter.js';
 const REVIEWS_HEADING = '## Reviews';
 export function parseReviewTable(planContent) {
     const reviewsSection = extractReviewsSection(planContent);
@@ -32,13 +33,23 @@ export async function scanUnprocessedReviews(projectPath) {
         throw error;
     }
 }
+export function parseReviewArtifactIdentity(content) {
+    const frontmatter = parseFrontmatterRecord(content);
+    const scope = typeof frontmatter.oat_review_scope === 'string'
+        ? frontmatter.oat_review_scope.trim()
+        : '';
+    const type = typeof frontmatter.oat_review_type === 'string'
+        ? frontmatter.oat_review_type.trim()
+        : '';
+    return scope && type ? { scope, type } : null;
+}
 function extractReviewsSection(planContent) {
-    const startIndex = planContent.indexOf(REVIEWS_HEADING);
-    if (startIndex === -1) {
+    const reviewsHeading = new RegExp(`^${REVIEWS_HEADING}[ \\t]*\\r?$`, 'm').exec(planContent);
+    if (!reviewsHeading) {
         return null;
     }
-    const remaining = planContent.slice(startIndex + REVIEWS_HEADING.length);
-    const nextHeadingIndex = remaining.search(/\n## /);
+    const remaining = planContent.slice(reviewsHeading.index + reviewsHeading[0].length);
+    const nextHeadingIndex = remaining.search(/^##(?!#)[ \t]+\S.*\r?$/m);
     if (nextHeadingIndex === -1) {
         return remaining.trim();
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@open-agent-toolkit/control-plane",
-  "version": "0.1.65",
+  "version": "0.1.69",
   "private": false,
   "description": "Read-only OAT control-plane library for structured project state",
   "homepage": "https://github.com/voxmedia/open-agent-toolkit/tree/main/packages/control-plane",