Skip to content

Run n fix#75

Merged
dpurandare merged 2 commits into
mainfrom
RunNFix
Dec 12, 2025
Merged

Run n fix#75
dpurandare merged 2 commits into
mainfrom
RunNFix

Conversation

@dpurandare

@dpurandare dpurandare commented Dec 12, 2025

Copy link
Copy Markdown
Owner

No description provided.

@dpurandare
docs: updated tomorrow plan with data dictionary tasks
58c3800
@dpurandare
feat: Implement Asset Hierarchy, Search, and Work Order Backend
8b2fdfc 
- **Asset Management (Sprint 1)**:
    - Added recursive asset hierarchy API.
    - Added tag management and advanced search (name, serial, tags).
    - Implemented circular reference protection.
- **Work Order Backend (Sprint 2)**:
    - Added parts, work_order_parts, work_order_labor tables.
    - Implemented WorkOrderStateMachine for strict lifecycle management.
    - Implemented comprehensive Work Order CRUD API with Tasks, Parts, and Labor support.
    - Added E2E tests for Assets (Hierarchy, Search) and Work Orders.
- **Documentation**:
    - Added Data Dictionary and Troubleshooting guide.
    - Updated walkthroughs.
@dpurandare dpurandare merged commit 99f9a9e into main Dec 12, 2025
8 of 27 checks passed
@dpurandare dpurandare deleted the RunNFix branch December 12, 2025 13:12
github-advanced-security[bot]
github-advanced-security AI found potential problems Dec 12, 2025
View reviewed changes
Comment thread backend/src/routes/work-orders.ts
import { ZodTypeProvider, serializerCompiler, validatorCompiler } from "fastify-type-provider-zod";
import { z } from "zod";
import { WorkOrderService } from "../services/work-order.service";
import { workOrderStatusEnum } from "../db/schema";

Check notice

Code scanning / CodeQL

Unused variable, import, function or class Note

Unused import workOrderStatusEnum.

Copilot Autofix

AI 7 months ago

To fix the problem, we should remove the unused import workOrderStatusEnum from the import statement on line 5. This will make the code easier to understand, avoid confusion, and eliminate any wasted parsing/import time. The only needed change is to modify the import statement so that it no longer imports workOrderStatusEnum. No additional imports or changes to other lines are required.

Suggested changeset 1
backend/src/routes/work-orders.ts

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/backend/src/routes/work-orders.ts b/backend/src/routes/work-orders.ts
--- a/backend/src/routes/work-orders.ts
+++ b/backend/src/routes/work-orders.ts
@@ -2,8 +2,8 @@
 import { ZodTypeProvider, serializerCompiler, validatorCompiler } from "fastify-type-provider-zod";
 import { z } from "zod";
 import { WorkOrderService } from "../services/work-order.service";
-import { workOrderStatusEnum } from "../db/schema";
 
+
 const WorkOrderStatusSchema = z.enum([
   "draft",
   "open",
EOF
@@ -2,8 +2,8 @@
import { ZodTypeProvider, serializerCompiler, validatorCompiler } from "fastify-type-provider-zod";
import { z } from "zod";
import { WorkOrderService } from "../services/work-order.service";
import { workOrderStatusEnum } from "../db/schema";


const WorkOrderStatusSchema = z.enum([
"draft",
"open",
Copilot is powered by AI and may make mistakes. Always verify output.
Comment thread backend/src/services/work-order.service.ts
sortOrder?: "asc" | "desc";
}
import { workOrders, workOrderTasks, workOrderParts, workOrderLabor } from "../db/schema";
import { eq, and, desc, asc, sql, like, or, inArray, gte, lte } from "drizzle-orm";

Check notice

Code scanning / CodeQL

Unused variable, import, function or class Note

Unused import inArray.

Copilot Autofix

AI 7 months ago

To fix the problem, simply remove the unused inArray import from the import statement on line 3. Make sure not to disturb the other imports from "drizzle-orm" that may still be used. This change only needs to happen on the line where the inArray is imported, and no further code changes are required.

Suggested changeset 1
backend/src/services/work-order.service.ts

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/backend/src/services/work-order.service.ts b/backend/src/services/work-order.service.ts
--- a/backend/src/services/work-order.service.ts
+++ b/backend/src/services/work-order.service.ts
@@ -1,6 +1,6 @@
 import { db } from "../db";
 import { workOrders, workOrderTasks, workOrderParts, workOrderLabor } from "../db/schema";
-import { eq, and, desc, asc, sql, like, or, inArray, gte, lte } from "drizzle-orm";
+import { eq, and, desc, asc, sql, like, or, gte, lte } from "drizzle-orm";
 import { WorkOrderStateMachine, WorkOrderStatus } from "./work-order-state";
 
 export interface CreateWorkOrderData {
EOF
@@ -1,6 +1,6 @@
import { db } from "../db";
import { workOrders, workOrderTasks, workOrderParts, workOrderLabor } from "../db/schema";
import { eq, and, desc, asc, sql, like, or, inArray, gte, lte } from "drizzle-orm";
import { eq, and, desc, asc, sql, like, or, gte, lte } from "drizzle-orm";
import { WorkOrderStateMachine, WorkOrderStatus } from "./work-order-state";

export interface CreateWorkOrderData {
Copilot is powered by AI and may make mistakes. Always verify output.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dpurandare @github-advanced-security