Replaces legacy adapter ids (codex_local / claude_local / gemini_local / claude_compatible) across ~42 test files with pi-mono equivalents: - codex_local → openai-codex - claude_local → claude-pro - gemini_local → gemini-cli - claude_compatible → anthropic Hand-rewrites where fixture semantics changed: - connector-matching: TASK_TYPE_PREFERENCE ordering rebased (claude-pro first for code/content/management; google first for research) - connector-model-policy: getModels(provider) from pi-ai instead of the deleted adapter.module.models array - session-management + edge-case-session: adapter exemption removed; rotation now uniform under pi-mono - ipc-core-admin-handlers: CSV compat model replaced with pi-ai anthropic - connectors.spec: lookup by openai-codex id (catalog now starts with claude-pro) - onboarding-model-pool: listProviderModels(provider) helper; label text changed to "ChatGPT Plus/Pro (Codex)" - database integration: added vault insecure-fallback env + electron mock - command-paths: uses new @main/connector-helpers normalizeCommandValue - pi-mono-migrations: added `as string` casts for 3 negative-existence assertions verifying legacy ids were removed Drops: - pnpm-workspace.yaml packages: [] (was "packages/*"; directory empty) - obsolete CONNECTOR_STRENGTHS import; .module.models probing Verification: - pnpm typecheck → 0 errors - pnpm lint → 0 errors / 0 warnings - pnpm build:verify → 0 errors / 0 warnings - pnpm test:unit → 508/508 pass - pnpm test:integration → 96/96 pass - All 6 adapter hard gates pass (NOT FOUND / 0 matches / empty) - pi-ai / pi-agent-core / pi-coding-agent all resolved @0.67.6
60 lines
1.7 KiB
TypeScript
60 lines
1.7 KiB
TypeScript
import { describe, expect, it } from "vitest";
|
|
import { hireRequestSchema } from "../../src/shared/contracts/schemas-agents";
|
|
import { getHireNameValidationError } from "../../src/main/agent-permissions";
|
|
|
|
const baseHireRequest = {
|
|
companyId: "00000000-0000-0000-0000-000000000000",
|
|
requestedByAgentId: null,
|
|
name: "Alice",
|
|
role: "Engineer",
|
|
title: "Backend Engineer",
|
|
department: "engineering" as const,
|
|
reportsTo: null,
|
|
connectorId: "openai-codex" as const,
|
|
workspaceId: null,
|
|
model: null,
|
|
metadataJson: "{}",
|
|
capabilities: "Build backend services",
|
|
budgetMonthlyUsd: 120,
|
|
};
|
|
|
|
describe("agent input schemas", () => {
|
|
it("rejects whitespace-only hire names", () => {
|
|
expect(() =>
|
|
hireRequestSchema.parse({
|
|
...baseHireRequest,
|
|
name: " ",
|
|
}),
|
|
).toThrow();
|
|
});
|
|
|
|
it("trims semantic hire fields before persistence", () => {
|
|
const parsed = hireRequestSchema.parse({
|
|
...baseHireRequest,
|
|
name: " Alice ",
|
|
role: " Engineer ",
|
|
title: " Backend Engineer ",
|
|
});
|
|
|
|
expect(parsed.name).toBe("Alice");
|
|
expect(parsed.role).toBe("Engineer");
|
|
expect(parsed.title).toBe("Backend Engineer");
|
|
});
|
|
|
|
it("rejects role-like names for non-executive hires", () => {
|
|
expect(getHireNameValidationError({
|
|
name: "Backend Engineer",
|
|
role: "Backend Engineer",
|
|
title: "Backend Engineer",
|
|
})).toBe("Hire name must be a real display name, not just the role.");
|
|
});
|
|
|
|
it("allows executive acronyms when they are the intended display name", () => {
|
|
expect(getHireNameValidationError({
|
|
name: "CTO",
|
|
role: "CTO",
|
|
title: "Chief Technology Officer",
|
|
})).toBeNull();
|
|
});
|
|
});
|