Release 1.2.2: FrankenPHP banner, release-check gates, LibreOffice locator PHPStan.

Author: HecFranco

.github/ISSUE_TEMPLATE/---bug-report.md

@@ -9,7 +9,7 @@ assignees: ''
 
 | Q | A |
 | ----------------------------| ----------------------- |
-| `word-template-bundle` version | x.y.z |
+| `word-to-pdf-bundle` version | x.y.z |
 | PHP version | x.y.z |
 | Symfony version | x.y |
 

.github/ISSUE_TEMPLATE/--support-question.md

@@ -9,4 +9,4 @@ assignees: ''
 
 ### Question
 
-### Versions (`composer info nowo-tech/word-template-bundle`, PHP, Symfony)
+### Versions (`composer info nowo-tech/word-to-pdf-bundle`, PHP, Symfony)

.github/workflows/ci.yml

@@ -71,6 +71,8 @@ jobs:
           composer require --dev symfony/framework-bundle:^${{ matrix.symfony }} symfony/yaml:^${{ matrix.symfony }} --no-interaction --prefer-dist --no-progress
 
       - name: PHPUnit (Symfony ${{ matrix.symfony }})
+        env:
+          SYMFONY_DEPRECATIONS_HELPER: max[direct]=0
         run: composer test
 
   tests:
@@ -102,9 +104,13 @@ jobs:
         run: composer phpstan
 
       - name: PHPUnit
+        env:
+          SYMFONY_DEPRECATIONS_HELPER: max[direct]=0
         run: composer test
 
       - name: PHPUnit coverage gate (min 100% lines)
+        env:
+          SYMFONY_DEPRECATIONS_HELPER: max[direct]=0
         run: composer coverage-check
 
       - name: Upload coverage to Codecov

.scripts/check-open-prs.sh

@@ -0,0 +1,113 @@
+#!/usr/bin/env bash
+# Fail if the GitHub repo has unresolved open pull requests (REQ-REL-003).
+# Allowed temporary exceptions: label hold|do-not-merge AND a future review-by ISO date
+# in the PR body (body must carry the date).
+set -euo pipefail
+
+ROOT="$(cd "$(dirname "$0")/.." && pwd -P)"
+cd "${ROOT}"
+
+if ! command -v gh >/dev/null 2>&1; then
+  echo "ERROR: gh CLI is required (REQ-REL-003)." >&2
+  exit 1
+fi
+
+if ! gh auth status >/dev/null 2>&1; then
+  echo "ERROR: gh is not authenticated — log in for nowo-tech (REQ-REL-003)." >&2
+  exit 1
+fi
+
+if ! command -v python3 >/dev/null 2>&1; then
+  echo "ERROR: python3 is required to evaluate open PR exceptions (REQ-REL-003)." >&2
+  exit 1
+fi
+
+# Resolve owner/name for -R (works when SSH remotes confuse `gh` default repo detection).
+resolve_repo() {
+  local url owner_repo
+  if url="$(git remote get-url origin 2>/dev/null)"; then
+    # git@github.com:org/repo.git  |  https://github.com/org/repo.git
+    owner_repo="$(printf '%s\n' "${url}" | sed -E 's#^(git@github\.com:|https://github\.com/)##; s#\.git$##')"
+    if [[ "${owner_repo}" == */* ]]; then
+      printf '%s\n' "${owner_repo}"
+      return 0
+    fi
+  fi
+  return 1
+}
+
+REPO_ARGS=()
+if REPO="$(resolve_repo)"; then
+  REPO_ARGS=(-R "${REPO}")
+fi
+
+PRS_JSON="$(gh pr list "${REPO_ARGS[@]}" --state open --limit 100 --json number,title,labels,body,url)"
+
+export PRS_JSON
+python3 <<'PY'
+import json
+import os
+import re
+import sys
+from datetime import date
+
+prs = json.loads(os.environ.get("PRS_JSON") or "[]")
+if not prs:
+    print("OK: no open pull requests (REQ-REL-003)")
+    sys.exit(0)
+
+HOLD_LABELS = {"hold", "do-not-merge"}
+REVIEW_BY_RE = re.compile(
+    r"(?i)\breview[-_ ]?by\b\s*[:=]?\s*(\d{4}-\d{2}-\d{2})"
+)
+
+today = date.today()
+unresolved = []
+held_ok = []
+
+for pr in prs:
+    labels = {str(l.get("name", "")).lower() for l in (pr.get("labels") or [])}
+    body = pr.get("body") or ""
+    m = REVIEW_BY_RE.search(body)
+    review_by = None
+    if m:
+        try:
+            review_by = date.fromisoformat(m.group(1))
+        except ValueError:
+            review_by = None
+
+    has_hold = bool(labels & HOLD_LABELS)
+    hold_valid = has_hold and review_by is not None and review_by >= today
+
+    entry = f"#{pr['number']} {pr.get('title') or ''} ({pr.get('url') or ''})"
+    if hold_valid:
+        held_ok.append(f"{entry} [hold until {review_by.isoformat()}]")
+    else:
+        reasons = []
+        if not has_hold:
+            reasons.append("missing hold/do-not-merge label")
+        if review_by is None:
+            reasons.append("missing review-by YYYY-MM-DD in body")
+        elif review_by < today:
+            reasons.append(f"review-by {review_by.isoformat()} expired")
+        unresolved.append(f"{entry} — {', '.join(reasons)}")
+
+if held_ok:
+    print("Allowed hold exceptions:")
+    for line in held_ok:
+        print(f"  {line}")
+
+if unresolved:
+    print("ERROR: unresolved open pull requests (REQ-REL-003):", file=sys.stderr)
+    for line in unresolved:
+        print(f"  {line}", file=sys.stderr)
+    print(
+        "Merge, close with reason, or label hold/do-not-merge with a future "
+        "review-by: YYYY-MM-DD in the PR body.",
+        file=sys.stderr,
+    )
+    sys.exit(1)
+
+print(f"OK: {len(held_ok)} open PR(s) covered by valid hold exceptions (REQ-REL-003)")
+sys.exit(0)
+PY

Makefile

@@ -1,8 +1,15 @@
 # WordToPdfBundle — Docker-driven development (REQ-MAKE-001 / REQ-MAKE-002)
-.PHONY: help up down build shell ensure-up install test test-coverage coverage-check cs-check cs-fix qa clean composer-sync release-check release-check-demos phpstan rector rector-dry update validate setup-hooks check-no-cursor-coauthor strip-cursor-coauthor-from-history assets update-deps
+.PHONY: help up down build shell ensure-up install test test-coverage coverage-check cs-check cs-fix qa clean composer-sync release-check release-check-demos demo-smoke phpstan rector rector-dry update validate setup-hooks check-no-cursor-coauthor check-open-prs strip-cursor-coauthor-from-history assets update-deps
 
 COMPOSE_FILE ?= docker-compose.yml
-COMPOSE      ?= docker-compose -f $(COMPOSE_FILE)
+# Prefer Compose V2; absolute docker path avoids shadowing by local docker/ (REQ-MAKE-010).
+DOCKER_BIN := $(shell PATH="/usr/local/bin:/usr/bin:/bin:$$PATH" command -v docker 2>/dev/null)
+ifeq ($(DOCKER_BIN),)
+COMPOSE_BIN ?= docker-compose
+else
+COMPOSE_BIN ?= $(shell $(DOCKER_BIN) compose version >/dev/null 2>&1 && echo "$(DOCKER_BIN) compose" || echo "docker-compose")
+endif
+COMPOSE     ?= $(COMPOSE_BIN) -f $(COMPOSE_FILE)
 SERVICE_PHP  ?= php
 COMPOSER_INSTALL = $(COMPOSE) exec -T $(SERVICE_PHP) sh -c 'composer install --no-interaction || { rm -rf vendor; composer clear-cache; composer install --no-interaction; }'
 DEMO_PRESENT := $(wildcard demo)
@@ -14,7 +21,8 @@ help:
 	@echo "  Dependencies: install, update, update-deps, composer-sync, validate"
 	@echo "  Tests: test, test-coverage, coverage-check"
 	@echo "  Quality: cs-check, cs-fix, rector, rector-dry, phpstan, qa"
-	@echo "  Release: release-check, release-check-demos"
+	@echo "  Release: release-check, release-check-demos, check-open-prs"
+	@echo "  demo-smoke      REQ-TEST-011: boot demo and assert HTTP 200"
 	@echo "  Demos: cd demo && make up   (or make -C demo/symfony8 up)"
 	@echo "  Assets: assets (no frontend assets in this bundle)"
 	@echo "  Git hooks: setup-hooks, check-no-cursor-coauthor"
@@ -85,6 +93,7 @@ assets:
 
 release-check: ensure-up
 	@$(MAKE) check-no-cursor-coauthor
+	@$(MAKE) check-open-prs
 	@$(MAKE) composer-sync
 	@$(MAKE) cs-fix
 	@$(MAKE) cs-check
@@ -98,6 +107,14 @@ endif
 release-check-demos:
 	@$(MAKE) -C demo release-check
 
+# REQ-TEST-011
+demo-smoke:
+	@$(MAKE) -C demo demo-smoke
+
+check-open-prs:
+	@chmod +x .scripts/check-open-prs.sh
+	@bash .scripts/check-open-prs.sh
+
 clean:
 	rm -rf vendor .phpunit.cache coverage .php-cs-fixer.cache coverage-php.txt coverage-output.txt
 
@@ -126,7 +143,8 @@ setup-hooks:
 
 # REQ-MAKE-008: update-deps
 BUNDLE_ROOT := $(abspath $(dir $(lastword $(MAKEFILE_LIST))))
-include $(BUNDLE_ROOT)/../.scripts/Makefile.update-deps.mk
+# Optional: monorepo helper absent on standalone GitHub Actions checkout (REQ-MAKE-009).
+-include $(BUNDLE_ROOT)/../.scripts/Makefile.update-deps.mk
 
 strip-cursor-coauthor-from-history:
 	@chmod +x .scripts/strip-cursor-coauthor-from-history.sh

README.md

@@ -1,66 +1,21 @@
 # WordToPdfBundle
 
-[![CI](https://github.com/nowo-tech/WordToPdfBundle/actions/workflows/ci.yml/badge.svg)](https://github.com/nowo-tech/WordToPdfBundle/actions/workflows/ci.yml)
-[![Packagist Version](https://img.shields.io/packagist/v/nowo-tech/word-to-pdf-bundle.svg?style=flat)](https://packagist.org/packages/nowo-tech/word-to-pdf-bundle)
-[![Packagist Downloads](https://img.shields.io/packagist/dt/nowo-tech/word-to-pdf-bundle.svg)](https://packagist.org/packages/nowo-tech/word-to-pdf-bundle)
-[![License](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
-[![PHP](https://img.shields.io/badge/PHP-8.2%2B-777BB4?logo=php)](https://php.net)
-[![Symfony](https://img.shields.io/badge/Symfony-7.4%20%7C%208.0%20%7C%208.1%2B-000000?logo=symfony)](https://symfony.com)
-[![GitHub stars](https://img.shields.io/github/stars/nowo-tech/word-to-pdf-bundle.svg?style=social&label=Star)](https://github.com/nowo-tech/WordToPdfBundle)
-[![Coverage](https://img.shields.io/badge/Coverage-100%25-brightgreen)](#tests-and-coverage)
+[![CI](https://github.com/nowo-tech/WordToPdfBundle/actions/workflows/ci.yml/badge.svg)](https://github.com/nowo-tech/WordToPdfBundle/actions/workflows/ci.yml) [![Packagist Version](https://img.shields.io/packagist/v/nowo-tech/word-to-pdf-bundle.svg?style=flat)](https://packagist.org/packages/nowo-tech/word-to-pdf-bundle) [![Packagist Downloads](https://img.shields.io/packagist/dt/nowo-tech/word-to-pdf-bundle.svg)](https://packagist.org/packages/nowo-tech/word-to-pdf-bundle) [![License](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE) [![PHP](https://img.shields.io/badge/PHP-8.2%2B-777BB4?logo=php)](https://php.net) [![Symfony](https://img.shields.io/badge/Symfony-7.4%20%7C%208.0%20%7C%208.1%2B-000000?logo=symfony)](https://symfony.com) [![GitHub stars](https://img.shields.io/github/stars/nowo-tech/word-to-pdf-bundle.svg?style=social&label=Star)](https://github.com/nowo-tech/WordToPdfBundle) [![Coverage](https://img.shields.io/badge/Coverage-100%25-brightgreen)](#tests-and-coverage)
 
 > **Found this useful?** Install from Packagist (`composer require nowo-tech/word-to-pdf-bundle`) and consider starring [WordToPdfBundle on GitHub](https://github.com/nowo-tech/WordToPdfBundle).
 
+**FrankenPHP demos:** runtime is selected with **`FRANKENPHP_MODE`** (`worker` default, or `classic` for per-request PHP / hot-reload). See [docs/DEMO-FRANKENPHP.md](docs/DEMO-FRANKENPHP.md).
 Symfony bundle that converts **Microsoft Word** (`.docx` / `.doc`) to **PDF** using **LibreOffice Writer** (`soffice` headless) for print-quality layout fidelity:
-
 - **named YAML profiles** + **default profile** + deep merge with per-call options, or **`convertWithInlineProfile()`**;
 - **batch conversion** via **`convertMany()`** with **`PdfNaming`** (keep / prefix / suffix / surround / fixed / callback, or path ⇒ filename map);
 - **runtime check** that **LibreOffice Writer** is installed (`nowo:word-to-pdf:check`); fails with install hints if missing;
 - works under **PHP-FPM** and **FrankenPHP** (Symfony Process / `proc_open`);
 - Symfony-friendly export: streamed/binary responses, local path, optional **Flysystem**.
-
 This bundle does **not** fill Word templates (see [WordTemplateBundle](https://github.com/nowo-tech/WordTemplateBundle)), convert HTML to Word (see [HtmlToWordBundle](https://github.com/nowo-tech/HtmlToWordBundle)), or use DomPDF (DomPDF cannot preserve Word styles).
 
-## Documentation
-
-- [Installation](docs/INSTALLATION.md)
-- [Configuration](docs/CONFIGURATION.md)
-- [Usage](docs/USAGE.md)
-- [Contributing](docs/CONTRIBUTING.md)
-- [Code of Conduct](CODE_OF_CONDUCT.md)
-- [Changelog](docs/CHANGELOG.md)
-- [Upgrading](docs/UPGRADING.md)
-- [Release](docs/RELEASE.md)
-- [Security](docs/SECURITY.md)
-- [Engram](docs/ENGRAM.md)
-- [Spec-driven development](docs/SPEC-DRIVEN-DEVELOPMENT.md)
-- [GitHub Spec Kit](docs/SPEC-KIT.md)
-
-### Additional documentation
-
-- [GitHub Actions CI requirements](docs/GITHUB_CI.md)
-- [FrankenPHP / Docker demo](docs/DEMO-FRANKENPHP.md) — `demo/symfony8` (see [`demo/README.md`](demo/README.md))
-
-## System requirement
-
-**LibreOffice Writer must be installed on the host / container** (Composer cannot install it):
-
-```bash
-# Debian / Ubuntu
-sudo apt-get install -y libreoffice-writer
-
-# Alpine
-apk add libreoffice
-
-# Fedora / RHEL
-sudo dnf install -y libreoffice-writer
-```
+![FrankenPHP Friendly Worker Mode](docs/images/frankenphp-friendly.png)
 
-Verify:
-
-```bash
-php bin/console nowo:word-to-pdf:check
-```
+This bundle is **FrankenPHP worker mode friendly**.
 
 ## Quick start
 
@@ -102,6 +57,34 @@ public function download(WordToPdfConverterInterface $converter, ExporterInterfa
 }
 ```
 
+## System requirement
+
+**LibreOffice Writer must be installed on the host / container** (Composer cannot install it):
+
+```bash
+# Debian / Ubuntu
+sudo apt-get install -y libreoffice-writer
+
+# Alpine
+apk add libreoffice
+
+# Fedora / RHEL
+sudo dnf install -y libreoffice-writer
+```
+
+Verify:
+
+```bash
+php bin/console nowo:word-to-pdf:check
+```
+
+## Development
+
+```bash
+make up
+make qa
+make release-check
+```
 ## FrankenPHP worker mode
 
 FrankenPHP worker mode: Supported (tested with LibreOffice conversion under FrankenPHP).
@@ -114,6 +97,26 @@ Demos use **`FRANKENPHP_MODE`** (`worker` by default, or `classic`) on PHP **8.5
 cd demo/symfony8 && cp .env.example .env && make up   # Symfony 8, port 8022
 ```
 
+## Documentation
+
+- [Installation](docs/INSTALLATION.md)
+- [Configuration](docs/CONFIGURATION.md)
+- [Usage](docs/USAGE.md)
+- [Contributing](docs/CONTRIBUTING.md)
+- [Code of Conduct](CODE_OF_CONDUCT.md)
+- [Changelog](docs/CHANGELOG.md)
+- [Upgrading](docs/UPGRADING.md)
+- [Release](docs/RELEASE.md)
+- [Security](docs/SECURITY.md)
+- [Engram](docs/ENGRAM.md)
+- [Spec-driven development](docs/SPEC-DRIVEN-DEVELOPMENT.md)
+- [GitHub Spec Kit](docs/SPEC-KIT.md)
+
+### Additional documentation
+
+- [GitHub Actions CI requirements](docs/GITHUB_CI.md)
+- [FrankenPHP / Docker demo](docs/DEMO-FRANKENPHP.md) — `demo/symfony8` (see [`demo/README.md`](demo/README.md))
+
 ## Tests and coverage
 
 | Scope | Detail |
@@ -127,10 +130,3 @@ composer test
 composer coverage-check
 ```
 
-## Development
-
-```bash
-make up
-make qa
-make release-check
-```

composer.json

@@ -6,6 +6,8 @@
     "keywords": [
         "symfony",
         "symfony-bundle",
+        "php",
+        "frankenphp",
         "word",
         "docx",
         "pdf",
@@ -16,7 +18,7 @@
     "homepage": "https://github.com/nowo-tech/WordToPdfBundle",
     "authors": [
         {
-            "name": "Héctor Franco Aceituno",
+            "name": "H\u00e9ctor Franco Aceituno",
             "email": "hectorfranco@nowo.tech"
         },
         {
@@ -37,6 +39,7 @@
     },
     "require-dev": {
         "friendsofphp/php-cs-fixer": "^3.0",
+        "nowo-tech/phpstan-frankenphp": "^1.0",
         "phpstan/extension-installer": "^1.0",
         "phpstan/phpstan": "^2.0",
         "phpstan/phpstan-phpunit": "^2.0",
@@ -89,5 +92,9 @@
         ],
         "rector": "@php vendor/bin/rector --no-progress-bar",
         "rector-dry": "@php vendor/bin/rector --dry-run --no-progress-bar"
+    },
+    "support": {
+        "issues": "https://github.com/nowo-tech/WordToPdfBundle/issues",
+        "source": "https://github.com/nowo-tech/WordToPdfBundle"
     }
 }