Move resources/ into postforge package for pip-installability

Resources (fonts, init scripts, encodings, device definitions) now live
inside the postforge/ package so they are included in wheels and sdists.
Path resolution uses package-relative lookups instead of os.chdir() to
the project root, preserving the user's working directory.

- git mv resources/ postforge/resources/
- Replace ProjectDir with PackageDir in system params
- Add package directory search in _resolve_filename()
- Remove os.chdir() from cli_runner.py
- VMDir uses tempfile.mkdtemp() (can't write into site-packages)
- Add resources/**/* to pyproject.toml package-data

Author: AndyCappDev

postforge/cli.py

@@ -71,12 +71,11 @@ def main() -> int:
 
     # Remember user's working directory — file paths on the command line
     # and in interactive mode should resolve relative to where pf was invoked
-    script_dir = os.path.dirname(os.path.abspath(__file__))
-    project_dir = os.path.dirname(script_dir)
+    package_dir = os.path.dirname(os.path.abspath(__file__))  # postforge/
     user_cwd = os.getcwd()
 
     # Get available devices from OutputDevice directory
-    device_dir = os.path.join(project_dir, "resources", "OutputDevice")
+    device_dir = os.path.join(package_dir, "resources", "OutputDevice")
 
     available_devices = []
     if os.path.exists(device_dir):
@@ -111,8 +110,7 @@ def main() -> int:
         print(f"System font cache rebuilt: {cache.font_count()} fonts found")
         return 0
 
-    # Resolve input file paths to absolute (relative to user's CWD) before
-    # we chdir to the project root for PS resource file resolution.
+    # Resolve input file paths to absolute (relative to user's CWD)
     inputfiles = [f if f == "-" else (os.path.join(user_cwd, f) if not os.path.isabs(f) else f)
                   for f in args.inputfiles]
     device = args.device
@@ -155,7 +153,7 @@ def main() -> int:
         inputfiles = [stdin_temp.name if f == "-" else f for f in inputfiles]
 
     try:
-        return run(args, inputfiles, stdin_temp, user_cwd, project_dir,
+        return run(args, inputfiles, stdin_temp, user_cwd, package_dir,
                    available_devices, device, memory_profile,
                    gc_analysis, leak_analysis, performance_profile,
                    profile_type, profile_output, page_filter)

postforge/cli_args.py

@@ -91,7 +91,7 @@ def get_output_base_name(outputfile: str, inputfiles: list) -> str:
 def _get_version() -> str:
     """Read the PostForge version from sysdict.ps (sole source of truth)."""
     sysdict = os.path.join(os.path.dirname(os.path.abspath(__file__)),
-                           os.pardir, "resources", "Init", "sysdict.ps")
+                           "resources", "Init", "sysdict.ps")
     with open(sysdict, "r") as f:
         for line in f:
             m = re.search(r'/revisionstring\s+\(([^)]+)\)', line)

postforge/cli_runner.py

@@ -222,8 +222,7 @@ def _conditional_paint(ctxt, elem):
     # Set output naming parameters in page device
     if ctxt.gstate.page_device:
         base_name = get_output_base_name(args.outputfile, inputfiles)
-        # Make output dir absolute relative to user's original CWD,
-        # since PS execution runs with CWD set to the project root.
+        # Make output dir absolute relative to user's CWD
         output_dir = args.output_dir
         if not os.path.isabs(output_dir):
             output_dir = os.path.join(user_cwd, output_dir)
@@ -478,7 +477,7 @@ def _run_interactive(ctxt, memory_profile, performance_profile, perf_profiler):
         print("\n" + ps_memory.generate_memory_report())
 
 
-def run(args, inputfiles, stdin_temp, user_cwd, project_dir,
+def run(args, inputfiles, stdin_temp, user_cwd, package_dir,
         available_devices, device, memory_profile, gc_analysis,
         leak_analysis, performance_profile, profile_type,
         profile_output, page_filter):
@@ -492,7 +491,7 @@ def run(args, inputfiles, stdin_temp, user_cwd, project_dir,
         inputfiles: List of resolved input file paths.
         stdin_temp: Temporary file for stdin input (or None).
         user_cwd: User's original working directory.
-        project_dir: PostForge project root directory.
+        package_dir: PostForge package directory (postforge/).
         available_devices: List of available device names.
         device: Explicitly requested device name (or None).
         memory_profile: Whether memory profiling is enabled.
@@ -529,20 +528,14 @@ def run(args, inputfiles, stdin_temp, user_cwd, project_dir,
     # Initialize the global system params
     system_params = init_system_params()
 
-    # Set CWD to project root for the entire session. PS resource files
-    # (fonts, devices, encodings) use relative paths like resources/Font/...
-    # that must resolve from the project root. Input file paths and output
-    # directories are made absolute (relative to user_cwd) before use.
-    os.chdir(project_dir)
     ctxt, err_string = create_context(system_params)
 
     if err_string:
         print(err_string)
         quit()
 
     # Store user's original CWD so file operators (run, file, etc.) can
-    # resolve relative paths against where the user invoked PostForge,
-    # not the project root we chdir'd to above.
+    # resolve relative paths against where the user invoked PostForge.
     ctxt.user_cwd = user_cwd
 
     # Take memory snapshot after context initialization

postforge/core/context_init.py

@@ -12,6 +12,7 @@
 import os
 import random
 import sys
+import tempfile
 import time
 from typing import Any, Dict, Optional, Tuple
 
@@ -35,33 +36,30 @@ def init_system_params() -> Dict[str, Any]:
 
     Returns:
         Dict[str, Any]: System parameters dictionary containing:
+            - PackageDir: Path to the postforge package directory
             - ByteOrder: Boolean indicating byte order (True for big-endian)
             - CurrDisplayList: Current display list index
             - FontResourceDir: Path to font resources
-            - GenericResourceDir: Path to generic resources
             - OutputDeviceResourceDir: Path to output device definitions
-            - VMDir: Path for virtual memory storage
+            - VMDir: Path for virtual memory storage (temp directory)
             - PageCount: Initial page count
             - PrinterName: Default printer name
             - RealFormat: Real number format ('IEE')
             - Revision: PostScript language revision level
     """
 
-    # Get the directory containing the postforge package
-    script_dir = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
-    # Get the parent directory (main project directory)
-    project_dir = os.path.dirname(script_dir)
+    # Get the postforge package directory (postforge/)
+    package_dir = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
 
     return {
-        "ProjectDir": project_dir,
+        "PackageDir": package_dir,
         "ByteOrder": True,
         "CurrDisplayList": 0,
-        "FontResourceDir": os.path.join(project_dir, "resources", "Font"),
-        "GenericResourceDir": os.path.join(project_dir, "generic_resources"),
+        "FontResourceDir": os.path.join(package_dir, "resources", "Font"),
         "OutputDeviceResourceDir": os.path.join(
-            project_dir, "resources", "OutputDevice"
+            package_dir, "resources", "OutputDevice"
         ),
-        "VMDir": os.path.join(project_dir, "vm"),
+        "VMDir": tempfile.mkdtemp(prefix="postforge_vm_"),
         "PageCount": 0,
         "PrinterName": "PostForge",
         "RealFormat": "IEE",
@@ -276,7 +274,7 @@ def create_context(
     #
     # add the filename to ctxt.strings
     # Use absolute path so PostForge works from any working directory
-    file_name = os.path.join(system_params["ProjectDir"], "resources", "Init", "sysdict.ps")
+    file_name = os.path.join(system_params["PackageDir"], "resources", "Init", "sysdict.ps")
     # Use forward slashes — backslashes are PS escape characters
     # (\r in \resources becomes carriage return, corrupting the path)
     file_name = file_name.replace("\\", "/")

postforge/operators/file.py

@@ -16,13 +16,12 @@
 def _resolve_filename(ctxt, filename):
     """Resolve a relative filename against likely directories.
 
-    PostForge changes CWD to the project root for internal resource resolution,
-    so relative paths in user PS files won't resolve against the user's directory.
-    This helper tries multiple locations in order:
+    Tries multiple locations in order:
 
-    1. As-is (works for absolute paths and project-relative resource files)
-    2. Relative to the directory of the currently executing file (e_stack walk)
-    3. Relative to the user's original working directory
+    1. As-is (works for absolute paths and files relative to CWD)
+    2. Relative to the package directory (resolves resources/Init/... etc.)
+    3. Relative to the directory of the currently executing file (e_stack walk)
+    4. Relative to the user's original working directory (safety net)
 
     Args:
         ctxt: PostScript execution context.
@@ -35,6 +34,13 @@ def _resolve_filename(ctxt, filename):
     if os.path.isabs(filename) or os.path.exists(filename):
         return filename
 
+    # Try relative to the package directory (where postforge/ lives)
+    package_dir = ctxt.system_params.get("PackageDir")
+    if package_dir:
+        candidate = os.path.join(package_dir, filename)
+        if os.path.exists(candidate):
+            return candidate
+
     # Try relative to the directory of the currently executing file
     for item in reversed(ctxt.e_stack):
         if isinstance(item, (ps.File, ps.Run)):