Switch to Nextcloud-major-aligned versioning starting at 34.0.0

Leading version number now tracks the supported Nextcloud major
(currently 34, i.e. Hub 26 Spring) instead of an independent 0.x
count, so compatibility is visible at a glance. Documents the scheme
in CLAUDE.md's release process section.

.gitea/workflows/build-main.yml

@@ -15,7 +15,7 @@ jobs:
     runs-on: gitea-runner-server03
     strategy:
       matrix:
-        php-version: ['8.2', '8.3', '8.4']
+        php-version: ['8.2', '8.3', '8.4', '8.5']
     steps:
       - uses: actions/checkout@v7
 

CLAUDE.md

@@ -109,6 +109,8 @@ There is no meaningful way to test the Deck integration itself outside a real Ne
 
 ### Release process
 
+**Versioning scheme (from 34.0.0 on): `<nextcloud-major>.<minor>.<patch>`.** The leading number tracks the minimum/maximum Nextcloud major version declared in `appinfo/info.xml`'s `<dependencies><nextcloud .../>` (currently `min-version="34" max-version="34"`), not this app's own feature history - so it only changes when the supported Nextcloud major changes (e.g. dropping/adding NC 34 support bumps to `35.0.0`), and resets `minor`/`patch` to `0.0` at that point. Ordinary fixes/features within the same supported NC major bump `minor`/`patch` as usual (semver-ish, without a leading `0.`). Versions before `34.0.0` (`0.1.x`/`0.2.x`) predate this scheme.
+
 1. Bump `<version>` in `appinfo/info.xml` and `version` in `package.json` (keep them in sync).
 2. Commit, push to `main`, confirm the run of `.gitea/workflows/build-main.yml` is green.
 3. `git tag vX.Y.Z && git push origin vX.Y.Z` — this triggers the tag branch of the same workflow, which re-runs the checks, builds the frontend, runs `make appstore`, and attaches `workflow_deck_automation.tar.gz` as the asset on a normal (non-pre-) release for that tag.

appinfo/info.xml

@@ -28,7 +28,7 @@ This app lets every user configure their own Deck automation workflows from Pers
 
 A background job periodically evaluates all enabled workflows and moves cards whose due date has passed, using Deck's own internal PHP classes only (no HTTP/OCS calls).
 ]]></description>
-    <version>0.2.0</version>
+    <version>34.0.0</version>
     <licence>agpl</licence>
     <author mail="patrick@niebel.ing">Patrick Niebeling</author>
     <namespace>WorkflowDeckAutomation</namespace>
@@ -36,7 +36,7 @@ A background job periodically evaluates all enabled workflows and moves cards wh
     <category>tools</category>
     <bugs>https://github.com/pniebeling/workflow_deck_automation/issues</bugs>
     <dependencies>
-        <php min-version="8.2" max-version="8.4"/>
+        <php min-version="8.2" max-version="8.5"/>
         <database min-version="9.4">pgsql</database>
         <database>sqlite</database>
         <database min-version="8.0">mysql</database>

lib/Service/DeckIntegrationService.php

@@ -483,7 +483,13 @@ class DeckIntegrationService {
 			$cardService = $this->resolve(CardService::class);
 
 			$cards = $cardMapper->findAllByStack($stackId);
-			$cards = $cardService->enrichCards($cards);
+			// enrichCards() mutates each $cards entry in place (labels, assigned users, ...) but
+			// *returns* CardDetails wrappers whose own fields (duedate, stackId, ...) were never
+			// copied from the wrapped card - only its overridden jsonSerialize() reads through to
+			// them. Capturing that return value here made every card's getDuedate()/getDaysUntilDue()
+			// resolve to null, so isOverdue() was always false and no workflow ever matched a card.
+			// Keep using the original, now-enriched $cards instead.
+			$cardService->enrichCards($cards);
 
 			return array_values(array_filter($cards, static function (Card $card) {
 				if (method_exists($card, 'getDeletedAt') && (int)$card->getDeletedAt() > 0) {

lib/Service/WorkflowRunner.php

@@ -176,6 +176,14 @@ class WorkflowRunner {
 				continue;
 			}
 
+			$this->logger->info('Card {card} ("{title}") matches workflow {id}; moving to stack {target}.', [
+				'app' => 'workflow_deck_automation',
+				'card' => $card->getId(),
+				'title' => $card->getTitle(),
+				'id' => $workflow->getId(),
+				'target' => $workflow->getTargetStackId(),
+			]);
+
 			if (!$this->moveCard($workflow, $card)) {
 				continue;
 			}
@@ -254,6 +262,14 @@ class WorkflowRunner {
 			return false;
 		}
 
+		$this->logger->info('Moved card {card} ("{title}") to stack {target} for workflow {id}.', [
+			'app' => 'workflow_deck_automation',
+			'card' => $card->getId(),
+			'title' => $card->getTitle(),
+			'target' => $workflow->getTargetStackId(),
+			'id' => $workflow->getId(),
+		]);
+
 		return true;
 	}
 

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "workflow_deck_automation",
-	"version": "0.2.0",
+	"version": "34.0.0",
 	"private": true,
 	"type": "module",
 	"description": "Automates moving overdue Deck cards between stacks, configurable per user.",