chore(release): v1.2.3 — accept category_id on transaction create

Create endpoint resolved category only by name and ignored numeric
category_id; native clients send category_id, so app-created
transactions were saved uncategorised. Create now honours category_id
as a fallback (validated against existing categories) and duplicate
detection resolves the name from the id. Web (name-based) unchanged.

Author: IT-BAER

CHANGELOG.md

@@ -2,6 +2,11 @@
 
 All notable changes to this project are documented in this file.
 
+## [v1.2.3] - 2026-06-06
+
+### Fixed
+- **Native-app transaction categories now persist on create**: the create endpoint previously resolved a category only from a `category` *name* and silently ignored a numeric `category_id`. Native clients send the category as a `category_id`, so every transaction created from the app was saved uncategorised — while edits were unaffected because the update endpoint already honoured `category_id`. Create now accepts `category_id` as a fallback when no name is supplied, validating that it refers to an existing category, and duplicate detection resolves the category name from that id so id-based creates are matched correctly. The web app (which sends the category name) is unchanged.
+
 ## [v1.2.2] - 2026-06-01
 
 ### Fixed

controllers/transactionController.js

@@ -13,6 +13,7 @@ const createTransaction = async (req, res) => {
   try {
     const {
       category,
+      category_id,
       source,
       target,
       amount,
@@ -36,8 +37,9 @@ const createTransaction = async (req, res) => {
     const inSource = norm(source);
     const inTarget = norm(target);
 
-    // Handle category (global by name)
+    // Handle category (global by name, or by numeric id from native clients)
     let categoryId = null;
+    let categoryNameForDup = inCategory;
     if (inCategory) {
       try {
         // Try to find any category with the same name regardless of owner
@@ -59,6 +61,20 @@ const createTransaction = async (req, res) => {
         console.error("Category handling error:", err);
         return res.status(500).json({ message: "Failed to process category" });
       }
+    } else if (category_id != null) {
+      // Native clients send the category as a numeric id (names are display-only).
+      // Accept the id directly when it refers to an existing category.
+      const idNum = Number(category_id);
+      if (Number.isInteger(idNum) && idNum > 0) {
+        const byId = await db.query(
+          "SELECT id, name FROM categories WHERE id = $1 LIMIT 1",
+          [idNum],
+        );
+        if (byId.rows.length > 0) {
+          categoryId = byId.rows[0].id;
+          categoryNameForDup = byId.rows[0].name;
+        }
+      }
     }
 
     // Handle source and target normally - let frontend manage the semantic mapping
@@ -129,7 +145,7 @@ const createTransaction = async (req, res) => {
       amount,
       transactionType,
       description || "",
-      inCategory || null,
+      categoryNameForDup || null,
       inSource || null,
       inTarget || null,
     ]);

package.json

@@ -1,6 +1,6 @@
 {
   "name": "finx",
-  "version": "1.2.2",
+  "version": "1.2.3",
   "description": "A modern personal finance management application",
   "main": "server.js",
   "scripts": {