merge: land axon-10629702 user-story tagging + coverage check [axon-10629702]

Author: easel

ui/package.json

@@ -13,7 +13,8 @@
 		"test:e2e": "cd .. && AXON_E2E_STORAGE=postgres scripts/test-ui-e2e-docker.sh",
 		"test:e2e:real": "cd .. && scripts/test-ui-e2e-docker.sh",
 		"test:e2e:postgres": "cd .. && AXON_E2E_STORAGE=postgres scripts/test-ui-e2e-docker.sh",
-		"test:e2e:sqlite": "cd .. && AXON_E2E_STORAGE=sqlite scripts/test-ui-e2e-docker.sh"
+		"test:e2e:sqlite": "cd .. && AXON_E2E_STORAGE=sqlite scripts/test-ui-e2e-docker.sh",
+		"check:story-coverage": "bun scripts/check-story-coverage.ts"
 	},
 	"devDependencies": {
 		"@biomejs/biome": "^1.9.4",

ui/scripts/check-story-coverage.ts

@@ -0,0 +1,72 @@
+#!/usr/bin/env bun
+/**
+ * check-story-coverage.ts
+ *
+ * Verifies that every FEAT-031 user story (US-113..US-119) has at least one
+ * Playwright test or describe block tagged with `@US-NNN` in ui/tests/e2e/.
+ *
+ * Usage:
+ *   bun scripts/check-story-coverage.ts
+ *
+ * Exits 0 if all stories are covered, non-zero otherwise.
+ */
+
+import { readdirSync, readFileSync } from 'node:fs';
+import { join } from 'node:path';
+
+const REQUIRED_STORIES = [113, 114, 115, 116, 117, 118, 119] as const;
+
+const E2E_DIR = join(import.meta.dir, '..', 'tests', 'e2e');
+
+// Collect all spec files
+const specFiles = readdirSync(E2E_DIR).filter((f) => f.endsWith('.spec.ts'));
+
+// Map: US number -> array of spec filenames that contain a tag
+const coverage = new Map<number, string[]>();
+for (const story of REQUIRED_STORIES) {
+	coverage.set(story, []);
+}
+
+const tagPattern = /@US-(\d+)/g;
+
+for (const file of specFiles) {
+	const content = readFileSync(join(E2E_DIR, file), 'utf-8');
+	const found = new Set<number>();
+	let match: RegExpExecArray | null;
+	tagPattern.lastIndex = 0;
+	while ((match = tagPattern.exec(content)) !== null) {
+		const num = parseInt(match[1], 10);
+		if (REQUIRED_STORIES.includes(num as (typeof REQUIRED_STORIES)[number])) {
+			found.add(num);
+		}
+	}
+	for (const num of found) {
+		coverage.get(num)?.push(file);
+	}
+}
+
+// Report
+console.log('Story coverage report (FEAT-031 / FEAT-015 / FEAT-016 / FEAT-029 / FEAT-030):');
+console.log('');
+
+let allCovered = true;
+for (const story of REQUIRED_STORIES) {
+	const specs = coverage.get(story) ?? [];
+	if (specs.length === 0) {
+		console.error(`  MISSING  US-${story}  — no tagged test found`);
+		allCovered = false;
+	} else {
+		console.log(`  COVERED  US-${story}  — ${specs.join(', ')}`);
+	}
+}
+
+console.log('');
+
+if (!allCovered) {
+	const uncovered = REQUIRED_STORIES.filter((s) => (coverage.get(s) ?? []).length === 0);
+	const label = uncovered.map((s) => `US-${s}`).join(', ');
+	console.error(`uncovered: ${label}`);
+	process.exit(1);
+}
+
+console.log('All FEAT-031 stories have at least one live-server workflow test tag.');

ui/tests/e2e/approval-inbox.spec.ts

@@ -4,9 +4,11 @@ import {
 	TASK_COLLECTION,
 	activateProposedPolicy,
 	approveIntent,
+	captureDataPlaneRequests,
 	createBudgetRecord,
 	dbIntentUrl,
 	dbIntentsUrl,
+	expectGraphqlPrimaryDataPlane,
 	patchBudgetRecordAs,
 	previewBudgetIntent,
 	proposedPolicyDraftDenyHigh,
@@ -21,12 +23,13 @@ async function selectStatus(page: Page, status: string) {
 }
 
 test.describe('Approval inbox', () => {
-	test('lists scoped intents across review states and opens detail', async ({ page, request }) => {
+	test('lists scoped intents across review states and opens detail @US-117', async ({ page, request }) => {
 		const db = await seedApprovalCollections(request, 'approval-inbox');
 		const ids = await seedIntentStates(request, db);
 		const foreignDb = await seedApprovalCollections(request, 'approval-foreign');
 		await createBudgetRecord(request, foreignDb, TASK_COLLECTION, 'task-foreign');
 		const foreign = await previewBudgetIntent(request, foreignDb, 'task-foreign', 20_000);
+		const requests = captureDataPlaneRequests(page, db);
 
 		await routeGraphqlAs(page, 'finance-approver');
 		await page.goto(dbIntentsUrl(db));
@@ -64,6 +67,8 @@ test.describe('Approval inbox', () => {
 		await expect(page.getByTestId('intent-audit-trail')).toContainText('intent.approve');
 		await expect(page.getByTestId('intent-audit-trail')).toContainText('approved');
 		await expect(page.getByTestId('intent-deep-links')).toContainText('Open audit log');
+
+		expectGraphqlPrimaryDataPlane(requests, 'approval inbox route should stay GraphQL-primary');
 	});
 
 	test('supports dense filters, keyboard selection, and inline review without leaving inbox', async ({
@@ -182,7 +187,7 @@ test.describe('Approval inbox', () => {
 		await expect(page.getByTestId('intent-reason')).toHaveValue('lost role attempt');
 	});
 
-	test('shows disabled action states for rejected, expired, committed, and stale intents', async ({
+	test('shows disabled action states for rejected, expired, committed, and stale intents @US-118', async ({
 		page,
 		request,
 	}) => {

ui/tests/e2e/graphql-policy-console.spec.ts

@@ -3,6 +3,8 @@ import {
 	SCN017_COLLECTIONS,
 	SCN017_ROLES,
 	SCN017_SUBJECTS,
+	captureDataPlaneRequests,
+	expectGraphqlPrimaryDataPlane,
 	routeGraphqlAs,
 	seedScn017PolicyUiFixture,
 } from './helpers';
@@ -12,9 +14,10 @@ function escapeRegExp(value: string): string {
 }
 
 test.describe('GraphQL policy console', () => {
-	test('opens an effectivePolicy preset from the policy workspace', async ({ page, request }) => {
+	test('opens an effectivePolicy preset from the policy workspace @US-113', async ({ page, request }) => {
 		const fixture = await seedScn017PolicyUiFixture(request, 'graphql-policy-console-effective');
 		const policiesUrl = `/ui/tenants/${encodeURIComponent(fixture.tenant.db_name)}/databases/${encodeURIComponent(fixture.db.name)}/policies`;
+		const requests = captureDataPlaneRequests(page, fixture.db);
 
 		await routeGraphqlAs(page, SCN017_SUBJECTS.contractor);
 		await page.goto(policiesUrl);
@@ -46,6 +49,8 @@ test.describe('GraphQL policy console', () => {
 		await expect(page.getByTestId('graphql-response')).toContainText('"canRead": true');
 		await expect(page.getByTestId('graphql-response')).toContainText('amount_cents');
 		await expect(page.getByTestId('graphql-response')).toContainText('commercial_terms');
+
+		expectGraphqlPrimaryDataPlane(requests, 'graphql policy console route should stay GraphQL-primary');
 	});
 
 	test('opens an explainPolicy preset from the policy workspace', async ({ page, request }) => {

ui/tests/e2e/intent-audit-lineage.spec.ts

@@ -6,11 +6,13 @@ import {
 	TASK_COLLECTION,
 	activateProposedPolicy,
 	approveIntent,
+	captureDataPlaneRequests,
 	commitIntent,
 	createBudgetRecord,
 	dbAuditUrl,
 	dbIntentUrl,
 	dbIntentsUrl,
+	expectGraphqlPrimaryDataPlane,
 	graphqlPath,
 	patchBudgetRecordAs,
 	previewBudgetIntent,
@@ -22,12 +24,13 @@ import {
 } from './helpers';
 
 test.describe('Intent audit lineage', () => {
-	test('shows delegated MCP intent metadata in the inbox and detail panels', async ({
+	test('shows delegated MCP intent metadata in the inbox and detail panels @US-119', async ({
 		page,
 		request,
 	}) => {
 		const db = await seedApprovalCollections(request, 'intent-audit-lineage');
 		const ids = await seedIntentStates(request, db);
+		const requests = captureDataPlaneRequests(page, db);
 
 		await routeGraphqlAs(page, 'finance-approver');
 		await page.goto(dbIntentsUrl(db));
@@ -68,9 +71,11 @@ test.describe('Intent audit lineage', () => {
 		await expect(page.getByTestId('intent-tool-arguments')).not.toContainText('23000');
 		await expect(page.getByTestId('intent-structured-outcome')).toContainText('needs_approval');
 		await expect(page.getByTestId('intent-structured-outcome')).toContainText('pending');
+
+		expectGraphqlPrimaryDataPlane(requests, 'intent audit lineage route should stay GraphQL-primary');
 	});
 
-	test('shows conflict outcomes for stale MCP-originated intent commits', async ({
+	test('shows conflict outcomes for stale MCP-originated intent commits @US-118', async ({
 		page,
 		request,
 	}) => {
@@ -148,7 +153,7 @@ test.describe('Intent audit lineage', () => {
 });
 
 test.describe('Intent audit deep link and lineage panel', () => {
-	test('deep link filters /audit by intent ID and pre-populates filter', async ({
+	test('deep link filters /audit by intent ID and pre-populates filter @US-116', async ({
 		page,
 		request,
 	}) => {

ui/tests/e2e/mcp-envelope-preview.spec.ts

@@ -17,7 +17,7 @@ function escapeRegExp(value: string): string {
 }
 
 test.describe('MCP envelope preview', () => {
-	test('mirrors explainPolicy outcomes for read, needs_approval, and denied flows', async ({
+	test('mirrors explainPolicy outcomes for read, needs_approval, and denied flows @US-119', async ({
 		page,
 		request,
 	}) => {

ui/tests/e2e/mutation-intents.spec.ts

@@ -4,8 +4,10 @@ import {
 	TASK_COLLECTION,
 	type TestDatabase,
 	activateProposedPolicy,
+	captureDataPlaneRequests,
 	dbCollectionUrl,
 	dbIntentUrl,
+	expectGraphqlPrimaryDataPlane,
 	patchBudgetRecordAs,
 	proposedPolicyDraftDenyHigh,
 	routeGraphqlAs,
@@ -52,11 +54,12 @@ async function previewIntent(page: Page): Promise<PreviewPayload> {
 }
 
 test.describe('Mutation intents', () => {
-	test('renders and commits an allowed mutation intent without showing the token', async ({
+	test('renders and commits an allowed mutation intent without showing the token @US-116', async ({
 		page,
 		request,
 	}) => {
 		const db = await seedIntentCollection(request, 'intent-allow');
+		const requests = captureDataPlaneRequests(page, db);
 		await routeGraphqlAs(page, 'finance-agent');
 		await openEntityEditor(page, db);
 		await fillJsonField(page, 'budget_cents', '6000');
@@ -77,6 +80,8 @@ test.describe('Mutation intents', () => {
 		await page.getByTestId('intent-commit').click();
 		await expect(page.getByText('Saved v2.')).toBeVisible({ timeout: 10_000 });
 		await expect(modal).toContainText('committed');
+
+		expectGraphqlPrimaryDataPlane(requests, 'mutation intents route should stay GraphQL-primary');
 	});
 
 	test('renders a needs-approval preview with approval route details', async ({
@@ -119,7 +124,7 @@ test.describe('Mutation intents', () => {
 		await expect(page.getByTestId('intent-commit')).toBeDisabled();
 	});
 
-	test('renders stale pre-image conflict details after preview drift', async ({
+	test('renders stale pre-image conflict details after preview drift @US-118', async ({
 		page,
 		request,
 	}) => {

ui/tests/e2e/policy-authoring.spec.ts

@@ -144,7 +144,7 @@ async function routeGraphqlWithDraft(
 }
 
 test.describe('Policy authoring', () => {
-	test('runs read, patch, and transaction policy evaluations from the workspace', async ({
+	test('runs read, patch, and transaction policy evaluations from the workspace @US-113 @US-114', async ({
 		page,
 		request,
 	}) => {
@@ -612,7 +612,7 @@ test.describe('Policy authoring (transaction-row delta)', () => {
 });
 
 test.describe('Policy authoring (schemas tab)', () => {
-	test('compile + fixture dry-run + activate updates the persisted policy', async ({
+	test('compile + fixture dry-run + activate updates the persisted policy @US-114', async ({
 		page,
 		request,
 	}) => {

ui/tests/e2e/policy-enforcement.spec.ts

@@ -4,11 +4,13 @@ import {
 	SCN017_COLLECTIONS,
 	SCN017_SUBJECTS,
 	activateProposedPolicy,
+	captureDataPlaneRequests,
 	createTestEntity,
 	createTestLink,
 	dbAuditUrl,
 	dbCollectionUrl,
 	dbCollectionsUrl,
+	expectGraphqlPrimaryDataPlane,
 	proposedPolicyDraftDenyHigh,
 	routeGraphqlAs,
 	seedScn017PolicyUiFixture,
@@ -57,14 +59,15 @@ function captureSubscribeFrame(page: Page): Promise<unknown> {
  */
 
 test.describe('Policy enforcement (UI redaction)', () => {
-	test('contractor sees redacted commercial_terms across list, detail, and audit views', async ({
+	test('contractor sees redacted commercial_terms across list, detail, and audit views @US-115', async ({
 		page,
 		request,
 	}) => {
 		const fixture = await seedScn017PolicyUiFixture(request, 'policy-enforcement-redaction');
 		const sensitiveCommercialTerms = 'net-15 expedited infrastructure terms';
 		const collectionUrl = dbCollectionUrl(fixture.db, SCN017_COLLECTIONS.invoices);
 		const auditUrl = dbAuditUrl(fixture.db);
+		const requests = captureDataPlaneRequests(page, fixture.db);
 
 		await routeGraphqlAs(page, SCN017_SUBJECTS.contractor);
 		await page.goto(collectionUrl);
@@ -112,6 +115,8 @@ test.describe('Policy enforcement (UI redaction)', () => {
 			.click({ trial: false });
 		const auditHtml = await page.content();
 		expect(auditHtml).not.toContain(sensitiveCommercialTerms);
+
+		expectGraphqlPrimaryDataPlane(requests, 'policy enforcement route should stay GraphQL-primary');
 	});
 
 	test('denied delete surfaces stable code, reason, and policy explanation', async ({