Stabilize build, API linkage, and Windows CI

Modernizes the project for the 0.6.0-pre-alpha milestone by tightening CMake configuration (3.23+, Windows-only guard, warning controls), removing stale/empty module wiring, and adding relocatable install/export packaging with `Pyramid::Engine` plus an external `find_package` consumer test. Adds a full Windows GitHub Actions matrix (Debug/Release) for configure/build/test/install/consumer validation, expands and fixes test coverage (including API linkage and corrected PNG/JPEG/zlib fixtures), and closes public-surface gaps by implementing or explicitly failing unsupported scene/texture methods while removing undeclared render-pass placeholders. Documentation and changelog were rewritten to match the current supported scope and workflow.

Author: OmarAglan

.github/workflows/windows-ci.yml

@@ -0,0 +1,70 @@
+name: Windows CI
+
+on:
+  push:
+    branches: [master, main]
+  pull_request:
+  workflow_dispatch:
+
+permissions:
+  contents: read
+
+jobs:
+  build-and-test:
+    runs-on: windows-2022
+    strategy:
+      fail-fast: false
+      matrix:
+        configuration: [Debug, Release]
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Configure
+        shell: pwsh
+        run: >-
+          cmake -S . -B build
+          -G "Visual Studio 17 2022"
+          -A x64
+          -DPYRAMID_BUILD_EXAMPLES=ON
+          -DPYRAMID_BUILD_TESTS=ON
+          -DPYRAMID_WARNINGS_AS_ERRORS=OFF
+
+      - name: Build
+        shell: pwsh
+        run: cmake --build build --config ${{ matrix.configuration }} --parallel
+
+      - name: Test
+        shell: pwsh
+        run: ctest --test-dir build -C ${{ matrix.configuration }} --output-on-failure
+
+      - name: Install smoke test
+        shell: pwsh
+        run: cmake --install build --config ${{ matrix.configuration }} --prefix build/install
+
+      - name: Configure external consumer
+        shell: pwsh
+        run: >-
+          cmake -S Tests/Consumer -B build/consumer
+          -G "Visual Studio 17 2022"
+          -A x64
+          -DCMAKE_PREFIX_PATH="${{ github.workspace }}/build/install"
+
+      - name: Build external consumer
+        shell: pwsh
+        run: cmake --build build/consumer --config ${{ matrix.configuration }} --parallel
+
+      - name: Run external consumer
+        shell: pwsh
+        run: ./build/consumer/${{ matrix.configuration }}/PyramidConsumer.exe
+
+      - name: Upload binaries
+        if: always()
+        uses: actions/upload-artifact@v4
+        with:
+          name: pyramid-${{ matrix.configuration }}
+          path: |
+            build/bin/${{ matrix.configuration }}
+            build/Examples/BasicRendering/${{ matrix.configuration }}
+          if-no-files-found: warn

AGENTS.md

@@ -1,59 +1,54 @@
 # Repository guidelines
 
-## Project structure
+## Scope
 
-- `Engine/` builds the `PyramidEngine` C++17 library.
-- Active modules use `include/` and `source/` directories: Core, Graphics, Math, Platform, and Utils.
-- Audio, Input, and Physics are placeholders and must not be described as implemented.
-- `Engine/Utils/test/` contains utility test executables.
-- `Examples/BasicGame` and `Examples/BasicRendering` are the current runnable examples.
-- `docs/` contains the maintained compact documentation set.
-- `vendor/glad/` is the bundled OpenGL/WGL loader.
-- Generated `build*` directories stay uncommitted.
+- `Engine/` builds the C++17 `PyramidEngine` library.
+- Active modules are Core, Graphics, Math, Platform, and Utils.
+- `Examples/BasicGame` and `Examples/BasicRendering` are the graphical references.
+- `Tests/PublicApiLinkage.cpp` protects selected public symbols.
+- `Tests/Consumer` validates the installed CMake package.
+- `vendor/glad` is a bundled public dependency.
+- Input, audio, physics, editor, scripting, DirectX, Vulkan, Linux, and macOS are not supported.
 
 ## Build and test
 
-Use the Windows/Visual Studio presets from the repository root:
-
-```powershell
-cmake --preset vs2022-debug
-cmake --build --preset build-debug
-```
-
-With tests:
-
 ```powershell
 cmake --preset vs2022-debug-tests
-cmake --build --preset build-debug-tests-clean
+cmake --build --preset build-debug-tests
 ctest --preset test-debug
+./scripts/run-smoke.ps1 -BuildDir build/debug-tests -Config Debug -DurationSeconds 5
 ```
 
-For graphics/runtime changes:
+Release validation:
 
 ```powershell
-./scripts/run-smoke.ps1 -BuildDir build -Config Debug -DurationSeconds 5
+cmake --preset vs2022-release-tests
+cmake --build --preset build-release-tests
+ctest --preset test-release
 ```
 
-The current source is Win32/OpenGL-specific. Do not present Linux, macOS, DirectX, or Vulkan as supported.
-
 ## Style
 
-- C++17, four-space indentation, braces on new lines.
-- Types and most public methods: `PascalCase`.
-- Locals and parameters: `camelCase`.
-- Fields: `m_` prefix.
-- Prefer RAII and explicit ownership.
-- Use `PYRAMID_LOG_*`, `PYRAMID_ASSERT`, and `PYRAMID_CORE_ASSERT` for diagnostics.
-- Avoid adding required interface methods with silent no-op defaults.
+- C++17, four spaces, braces on new lines.
+- Types/public methods use `PascalCase`; locals/parameters use `camelCase`; fields use `m_`.
+- Prefer RAII, explicit ownership, and `PYRAMID_LOG_*` diagnostics.
+- Do not add required interface methods with silent no-op defaults.
+- Do not expose source-tree absolute paths through installed target interfaces.
+
+## Public APIs
+
+Every public declaration must be implemented, removed, or documented as an explicit failure. Add linkage-sensitive symbols to `Tests/PublicApiLinkage.cpp`.
+
+Do not describe placeholder algorithms as complete. Current examples and engine shaders target GLSL 3.30; the runtime requires OpenGL 3.3 core or newer.
 
 ## Tests
 
-Add focused `Test<Feature>.cpp` executables and register them with CTest. Tests must return non-zero on failure and print actionable context. Renderer changes also require visual inspection; the smoke script only detects early process failure.
+Tests must fail visibly, use valid fixtures, avoid false-success skips, clean temporary files, and print actionable context. Renderer changes require visual inspection because process smoke testing is not pixel validation.
 
 ## Documentation
 
-Update an existing maintained document rather than adding overlapping guides or status reports. Verify paths, target names, namespaces, signatures, platform requirements, and implementation status against source. Planned behavior belongs in `docs/ROADMAP.md`, not `docs/API.md`.
+Update the maintained compact set instead of adding overlapping guides or status files. Planned work belongs in `docs/ROADMAP.md`; historical changes belong in `CHANGELOG.md`.
 
 ## Pull requests
 
-Use concise imperative commit subjects. Include the affected modules, design/lifetime decisions, exact build and test commands, visible-output evidence for rendering changes, known limitations, and documentation/changelog updates.
+Include affected modules, ownership/API decisions, exact validation commands, visible rendering evidence, known limitations, and documentation/changelog updates.