feat(projects): auto-create NAS folder for order-spawned projects
Manual project creation already creates the 02_PROJEKTY/<number>_<name> folder (projects.service.createProject). Projects created from an order — both from an accepted offer (createOrderFromQuotation) and a manual offer-less order (createOrder) — did not, so they had no NAS folder until a user opened the project and clicked "create folder" or uploaded a file. - orders.service: best-effort, non-fatal createProjectFolder after each project-creating transaction commits (mirrors createProject; gated on isConfigured(), logs on failure, never rolls back the order). - projects.service: log when the existing auto-create returns false (was silently discarded — CLAUDE.md known-issue #4). - nas-file-manager: optional constructor basePath seam for tests. - tests: NasFileManager.createProjectFolder coverage — naming, idempotency, Czech diacritics, empty name, not-mounted (152 -> 158 tests). Bump version to 2.0.3. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "app-ts",
|
"name": "app-ts",
|
||||||
"version": "2.0.2",
|
"version": "2.0.3",
|
||||||
"description": "",
|
"description": "",
|
||||||
"main": "dist/server.js",
|
"main": "dist/server.js",
|
||||||
"scripts": {
|
"scripts": {
|
||||||
|
|||||||
70
src/__tests__/nas-project-folder.test.ts
Normal file
70
src/__tests__/nas-project-folder.test.ts
Normal file
@@ -0,0 +1,70 @@
|
|||||||
|
import { describe, it, expect, beforeEach, afterEach } from "vitest";
|
||||||
|
import fs from "fs";
|
||||||
|
import os from "os";
|
||||||
|
import path from "path";
|
||||||
|
import { NasFileManager } from "../services/nas-file-manager";
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Folder auto-creation primitive used by both the manual project-create path
|
||||||
|
* (projects.service createProject) and the order→project paths (orders.service
|
||||||
|
* createOrder / createOrderFromQuotation). Uses an isolated temp dir via the
|
||||||
|
* constructor seam so it is deterministic and passes even where the real NAS
|
||||||
|
* (Z:) is not mounted.
|
||||||
|
*/
|
||||||
|
describe("NasFileManager.createProjectFolder", () => {
|
||||||
|
let tmpBase: string;
|
||||||
|
let nas: NasFileManager;
|
||||||
|
|
||||||
|
beforeEach(() => {
|
||||||
|
tmpBase = fs.mkdtempSync(path.join(os.tmpdir(), "nas-proj-"));
|
||||||
|
nas = new NasFileManager(tmpBase);
|
||||||
|
});
|
||||||
|
|
||||||
|
afterEach(() => {
|
||||||
|
fs.rmSync(tmpBase, { recursive: true, force: true });
|
||||||
|
});
|
||||||
|
|
||||||
|
it("creates a <number>_<name> folder for a new project", () => {
|
||||||
|
const ok = nas.createProjectFolder("26710001", "Rekonstrukce haly");
|
||||||
|
expect(ok).toBe(true);
|
||||||
|
expect(
|
||||||
|
fs.existsSync(path.join(tmpBase, "26710001_Rekonstrukce_haly")),
|
||||||
|
).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("is idempotent — a second call does not create a duplicate", () => {
|
||||||
|
expect(nas.createProjectFolder("26710002", "Hala")).toBe(true);
|
||||||
|
expect(nas.createProjectFolder("26710002", "Hala")).toBe(true);
|
||||||
|
const matches = fs
|
||||||
|
.readdirSync(tmpBase)
|
||||||
|
.filter((e) => e.startsWith("26710002_"));
|
||||||
|
expect(matches).toEqual(["26710002_Hala"]);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("preserves Czech diacritics (no transliteration)", () => {
|
||||||
|
expect(nas.createProjectFolder("26710003", "Měření haly")).toBe(true);
|
||||||
|
expect(fs.existsSync(path.join(tmpBase, "26710003_Měření_haly"))).toBe(
|
||||||
|
true,
|
||||||
|
);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("strips illegal characters and collapses spaces", () => {
|
||||||
|
expect(nas.createProjectFolder("26710004", "A / B: C")).toBe(true);
|
||||||
|
expect(fs.existsSync(path.join(tmpBase, "26710004_A_B_C"))).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("handles an empty name without crashing (trailing underscore)", () => {
|
||||||
|
expect(nas.createProjectFolder("26710005", "")).toBe(true);
|
||||||
|
expect(fs.existsSync(path.join(tmpBase, "26710005_"))).toBe(true);
|
||||||
|
});
|
||||||
|
|
||||||
|
it("returns false and creates nothing when the NAS base is not mounted", () => {
|
||||||
|
const missing = new NasFileManager(
|
||||||
|
path.join(tmpBase, "does-not-exist-subdir"),
|
||||||
|
);
|
||||||
|
expect(missing.createProjectFolder("26710006", "X")).toBe(false);
|
||||||
|
expect(fs.existsSync(path.join(tmpBase, "does-not-exist-subdir"))).toBe(
|
||||||
|
false,
|
||||||
|
);
|
||||||
|
});
|
||||||
|
});
|
||||||
@@ -105,8 +105,12 @@ interface DownloadResult {
|
|||||||
export class NasFileManager {
|
export class NasFileManager {
|
||||||
private readonly basePath: string;
|
private readonly basePath: string;
|
||||||
|
|
||||||
constructor() {
|
constructor(basePath?: string) {
|
||||||
this.basePath = path.resolve(config.nas.path).replace(/\\/g, "/");
|
// basePath is an injection seam for tests; production callers pass nothing
|
||||||
|
// and get the configured NAS_PATH.
|
||||||
|
this.basePath = path
|
||||||
|
.resolve(basePath ?? config.nas.path)
|
||||||
|
.replace(/\\/g, "/");
|
||||||
}
|
}
|
||||||
|
|
||||||
public isConfigured(): boolean {
|
public isConfigured(): boolean {
|
||||||
|
|||||||
@@ -5,6 +5,11 @@ import {
|
|||||||
releaseSharedNumber,
|
releaseSharedNumber,
|
||||||
isOrderNumberTaken,
|
isOrderNumberTaken,
|
||||||
} from "./numbering.service";
|
} from "./numbering.service";
|
||||||
|
import { NasFileManager } from "./nas-file-manager";
|
||||||
|
|
||||||
|
// Best-effort NAS folder creation for order-spawned projects, mirroring
|
||||||
|
// projects.service. Stateless singleton; reads NAS_PATH at construction.
|
||||||
|
const nasFileManager = new NasFileManager();
|
||||||
|
|
||||||
interface OrderItemInput {
|
interface OrderItemInput {
|
||||||
description?: string | null;
|
description?: string | null;
|
||||||
@@ -262,6 +267,23 @@ export async function createOrderFromQuotation(
|
|||||||
return { order, project, orderNumber };
|
return { order, project, orderNumber };
|
||||||
});
|
});
|
||||||
|
|
||||||
|
// Best-effort NAS project folder, outside the transaction (mirrors
|
||||||
|
// createProject). A project created from an accepted offer must get the same
|
||||||
|
// 02_PROJEKTY/<number>_<name> folder as a manually-created one. Non-fatal: a
|
||||||
|
// NAS outage must never roll back the order.
|
||||||
|
if (result.project.project_number && nasFileManager.isConfigured()) {
|
||||||
|
const created = nasFileManager.createProjectFolder(
|
||||||
|
result.project.project_number,
|
||||||
|
result.project.name || "",
|
||||||
|
);
|
||||||
|
if (!created) {
|
||||||
|
console.error(
|
||||||
|
"[orders.service] NAS folder not created for project",
|
||||||
|
result.project.project_number,
|
||||||
|
);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
return {
|
return {
|
||||||
data: {
|
data: {
|
||||||
order_id: result.order.id,
|
order_id: result.order.id,
|
||||||
@@ -297,7 +319,7 @@ export async function createOrder(
|
|||||||
attachmentName?: string | null,
|
attachmentName?: string | null,
|
||||||
) {
|
) {
|
||||||
try {
|
try {
|
||||||
return await prisma.$transaction(async (tx) => {
|
const result = await prisma.$transaction(async (tx) => {
|
||||||
const orderNumber =
|
const orderNumber =
|
||||||
body.order_number !== undefined && body.order_number !== null
|
body.order_number !== undefined && body.order_number !== null
|
||||||
? String(body.order_number)
|
? String(body.order_number)
|
||||||
@@ -362,6 +384,8 @@ export async function createOrder(
|
|||||||
}
|
}
|
||||||
|
|
||||||
let projectId: number | undefined;
|
let projectId: number | undefined;
|
||||||
|
let createdProject: { project_number: string; name: string } | null =
|
||||||
|
null;
|
||||||
// Only auto-create a project for genuine offer-less orders; share the
|
// Only auto-create a project for genuine offer-less orders; share the
|
||||||
// order's number (same shared pool) exactly like the from-quotation flow.
|
// order's number (same shared pool) exactly like the from-quotation flow.
|
||||||
if (body.create_project !== false && body.quotation_id == null) {
|
if (body.create_project !== false && body.quotation_id == null) {
|
||||||
@@ -375,14 +399,42 @@ export async function createOrder(
|
|||||||
},
|
},
|
||||||
});
|
});
|
||||||
projectId = project.id;
|
projectId = project.id;
|
||||||
|
if (project.project_number) {
|
||||||
|
createdProject = {
|
||||||
|
project_number: project.project_number,
|
||||||
|
name: project.name || "",
|
||||||
|
};
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
return {
|
return {
|
||||||
id: order.id,
|
id: order.id,
|
||||||
order_number: order.order_number,
|
order_number: order.order_number,
|
||||||
project_id: projectId,
|
project_id: projectId,
|
||||||
|
createdProject,
|
||||||
};
|
};
|
||||||
});
|
});
|
||||||
|
|
||||||
|
// Best-effort NAS project folder, outside the transaction (mirrors
|
||||||
|
// createProject). Non-fatal so a NAS outage can't roll back the order.
|
||||||
|
if (result.createdProject && nasFileManager.isConfigured()) {
|
||||||
|
const created = nasFileManager.createProjectFolder(
|
||||||
|
result.createdProject.project_number,
|
||||||
|
result.createdProject.name,
|
||||||
|
);
|
||||||
|
if (!created) {
|
||||||
|
console.error(
|
||||||
|
"[orders.service] NAS folder not created for project",
|
||||||
|
result.createdProject.project_number,
|
||||||
|
);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return {
|
||||||
|
id: result.id,
|
||||||
|
order_number: result.order_number,
|
||||||
|
project_id: result.project_id,
|
||||||
|
};
|
||||||
} catch (err) {
|
} catch (err) {
|
||||||
if (err instanceof Error && "status" in err) {
|
if (err instanceof Error && "status" in err) {
|
||||||
return {
|
return {
|
||||||
|
|||||||
@@ -189,10 +189,16 @@ export async function createProject(data: CreateProjectInput) {
|
|||||||
});
|
});
|
||||||
|
|
||||||
if (project.project_number && nasFileManager.isConfigured()) {
|
if (project.project_number && nasFileManager.isConfigured()) {
|
||||||
nasFileManager.createProjectFolder(
|
const created = nasFileManager.createProjectFolder(
|
||||||
project.project_number,
|
project.project_number,
|
||||||
project.name || "",
|
project.name || "",
|
||||||
);
|
);
|
||||||
|
if (!created) {
|
||||||
|
console.error(
|
||||||
|
"[projects.service] NAS folder not created for project",
|
||||||
|
project.project_number,
|
||||||
|
);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
return project;
|
return project;
|
||||||
|
|||||||
Reference in New Issue
Block a user