fix: restructure for Mason submodule conventions and resolve warnings

  - Move submodules to src/quickchpl/ per Mason package structure
  - Add include module statements before public use in quickchpl.chpl
  - Fix Property generic type warnings with Property(?) syntax
  - Replace _unused loop variables with bare for loops
  - Update test imports to use quickchpl module
  - Fix Mason.toml trailing newline
  - Update CI paths for new module structure
  - Bump version to 1.0.2

  Resolves Mason "Main Module Check" failure and eliminates
  compiler warnings for generic types and unstable symbols.
  EOF

Author: OutBot CI

.github/workflows/ci.yml

@@ -30,7 +30,7 @@ jobs:
       - name: Verify library compiles
         run: |
           # Verify all modules compile together
-          chpl -M src --no-codegen src/quickchpl.chpl
+          chpl -M src -M src/quickchpl --no-codegen src/quickchpl.chpl
           echo "✓ All modules compile successfully"
 
   test-unit:
@@ -47,19 +47,19 @@ jobs:
       - name: Run Generator Tests
         run: |
           echo "Running Generator Tests..."
-          chpl test/unit/GeneratorTests.chpl src/*.chpl -o /tmp/generator_tests
+          chpl -M src -M src/quickchpl test/unit/GeneratorTests.chpl -o /tmp/generator_tests
           /tmp/generator_tests
 
       - name: Run Shrinker Tests
         run: |
           echo "Running Shrinker Tests..."
-          chpl test/unit/ShrinkerTests.chpl src/*.chpl -o /tmp/shrinker_tests
+          chpl -M src -M src/quickchpl test/unit/ShrinkerTests.chpl -o /tmp/shrinker_tests
           /tmp/shrinker_tests
 
       - name: Run Property Tests
         run: |
           echo "Running Property Tests..."
-          chpl test/unit/PropertyTests.chpl src/*.chpl -o /tmp/property_tests
+          chpl -M src -M src/quickchpl test/unit/PropertyTests.chpl -o /tmp/property_tests
           /tmp/property_tests
 
   test-examples:
@@ -76,19 +76,19 @@ jobs:
       - name: Run Getting Started Example
         run: |
           echo "Running Getting Started Example..."
-          chpl examples/GettingStarted.chpl src/*.chpl -o /tmp/getting_started
+          chpl -M src -M src/quickchpl examples/GettingStarted.chpl -o /tmp/getting_started
           /tmp/getting_started
 
       - name: Run Algebraic Properties Example
         run: |
           echo "Running Algebraic Properties Example..."
-          chpl examples/AlgebraicProperties.chpl src/*.chpl -o /tmp/algebraic
+          chpl -M src -M src/quickchpl examples/AlgebraicProperties.chpl -o /tmp/algebraic
           /tmp/algebraic
 
       - name: Run Custom Generators Example
         run: |
           echo "Running Custom Generators Example..."
-          chpl examples/CustomGenerators.chpl src/*.chpl -o /tmp/custom_gen
+          chpl -M src -M src/quickchpl examples/CustomGenerators.chpl -o /tmp/custom_gen
           /tmp/custom_gen
 
   test-self:
@@ -104,7 +104,7 @@ jobs:
 
       - name: Run quickchpl self-tests
         run: |
-          chpl test/properties/SelfTests.chpl src/*.chpl -o /tmp/self_tests
+          chpl -M src -M src/quickchpl test/properties/SelfTests.chpl -o /tmp/self_tests
           /tmp/self_tests --numTests=${{ env.QUICKCHPL_NUM_TESTS }}
 
   test-matrix:
@@ -124,7 +124,7 @@ jobs:
       - name: Build with Chapel ${{ matrix.chapel-version }}
         run: |
           chpl --version
-          chpl test/unit/GeneratorTests.chpl src/*.chpl -o /tmp/test
+          chpl -M src -M src/quickchpl test/unit/GeneratorTests.chpl -o /tmp/test
           /tmp/test
 
   lint:
@@ -140,7 +140,7 @@ jobs:
         run: |
           echo "Running chplcheck linter..."
           chplcheck --version || echo "chplcheck version unknown"
-          chplcheck src/*.chpl test/*.chpl examples/*.chpl > chplcheck.log 2>&1 || true
+          chplcheck src/*.chpl src/quickchpl/*.chpl test/*.chpl test/**/*.chpl examples/*.chpl > chplcheck.log 2>&1 || true
           cat chplcheck.log
 
           # Count violations (excluding intentional API design decisions)
@@ -152,9 +152,8 @@ jobs:
 
           # Known acceptable violations:
           # - 8 CamelCaseRecords (generator types - intentional API)
-          # - 21 UnusedLoopIndex (_unused variable - Chapel limitation)
           # - 1 PascalCaseModules (quickchpl - package name)
-          ACCEPTABLE=30
+          ACCEPTABLE=9
 
           if [ "$VIOLATIONS" -gt "$ACCEPTABLE" ]; then
             echo "❌ Too many chplcheck violations (expected <=$ACCEPTABLE, got $VIOLATIONS)"
@@ -166,7 +165,7 @@ jobs:
       - name: Check for trailing whitespace
         run: |
           echo "Checking for trailing whitespace..."
-          ! grep -rn '[[:space:]]$' src/*.chpl test/*.chpl examples/*.chpl || {
+          ! grep -rn '[[:space:]]$' src/*.chpl src/quickchpl/*.chpl test/*.chpl test/**/*.chpl examples/*.chpl || {
             echo "Found trailing whitespace"
             exit 1
           }
@@ -190,10 +189,9 @@ jobs:
 
       - name: Generate documentation with chpldoc
         run: |
-          cd src
-          for file in *.chpl; do
+          for file in src/quickchpl.chpl src/quickchpl/*.chpl; do
             echo "Generating docs for $file..."
-            chpldoc "$file" --output-dir ../docs/api || echo "Warning: chpldoc failed for $file"
+            chpldoc "$file" --output-dir docs/api || echo "Warning: chpldoc failed for $file"
           done
 
       - name: Upload documentation artifacts

.gitlab-ci.yml

@@ -29,7 +29,7 @@ build:
   script:
     - chpl --version
     # Verify all modules compile together
-    - chpl -M src --no-codegen src/quickchpl.chpl
+    - chpl -M src -M src/quickchpl --no-codegen src/quickchpl.chpl
     - echo "✓ All modules compile successfully"
 
 # ============================================
@@ -41,7 +41,7 @@ lint:chplcheck:
   script:
     - echo "Running chplcheck linter..."
     - chplcheck --version || echo "chplcheck version unknown"
-    - chplcheck src/*.chpl test/*.chpl examples/*.chpl > chplcheck.log 2>&1 || true
+    - chplcheck src/*.chpl src/quickchpl/*.chpl test/*.chpl test/**/*.chpl examples/*.chpl > chplcheck.log 2>&1 || true
     - cat chplcheck.log
     - |
       # Count violations (excluding intentional API design decisions)
@@ -53,9 +53,8 @@ lint:chplcheck:
 
       # Known acceptable violations:
       # - 8 CamelCaseRecords (generator types - intentional API)
-      # - 21 UnusedLoopIndex (_unused variable - Chapel limitation)
       # - 1 PascalCaseModules (quickchpl - package name)
-      ACCEPTABLE=30
+      ACCEPTABLE=9
 
       if [ "$VIOLATIONS" -gt "$ACCEPTABLE" ]; then
         echo "❌ Too many chplcheck violations (expected <=$ACCEPTABLE, got $VIOLATIONS)"
@@ -74,7 +73,7 @@ lint:whitespace:
   script:
     - echo "Checking for trailing whitespace..."
     - |
-      ! grep -rn '[[:space:]]$' src/*.chpl test/*.chpl examples/*.chpl || {
+      ! grep -rn '[[:space:]]$' src/*.chpl src/quickchpl/*.chpl test/*.chpl test/**/*.chpl examples/*.chpl || {
         echo "Found trailing whitespace"
         exit 1
       }
@@ -97,49 +96,49 @@ test:unit:generators:
   stage: test
   script:
     - echo "Running Generator Tests..."
-    - chpl test/unit/GeneratorTests.chpl src/*.chpl -o /tmp/generator_tests
+    - chpl -M src -M src/quickchpl test/unit/GeneratorTests.chpl -o /tmp/generator_tests
     - /tmp/generator_tests
 
 test:unit:shrinkers:
   stage: test
   script:
     - echo "Running Shrinker Tests..."
-    - chpl test/unit/ShrinkerTests.chpl src/*.chpl -o /tmp/shrinker_tests
+    - chpl -M src -M src/quickchpl test/unit/ShrinkerTests.chpl -o /tmp/shrinker_tests
     - /tmp/shrinker_tests
 
 test:unit:properties:
   stage: test
   script:
     - echo "Running Property Tests..."
-    - chpl test/unit/PropertyTests.chpl src/*.chpl -o /tmp/property_tests
+    - chpl -M src -M src/quickchpl test/unit/PropertyTests.chpl -o /tmp/property_tests
     - /tmp/property_tests
 
 test:examples:
   stage: test
   script:
     - echo "Running Getting Started Example..."
-    - chpl examples/GettingStarted.chpl src/*.chpl -o /tmp/getting_started
+    - chpl -M src -M src/quickchpl examples/GettingStarted.chpl -o /tmp/getting_started
     - /tmp/getting_started
     - echo "Running Algebraic Properties Example..."
-    - chpl examples/AlgebraicProperties.chpl src/*.chpl -o /tmp/algebraic
+    - chpl -M src -M src/quickchpl examples/AlgebraicProperties.chpl -o /tmp/algebraic
     - /tmp/algebraic
     - echo "Running Custom Generators Example..."
-    - chpl examples/CustomGenerators.chpl src/*.chpl -o /tmp/custom_gen
+    - chpl -M src -M src/quickchpl examples/CustomGenerators.chpl -o /tmp/custom_gen
     - /tmp/custom_gen
 
 test:self:
   stage: test
   script:
     - echo "Running quickchpl self-tests..."
-    - chpl test/properties/SelfTests.chpl src/*.chpl -o /tmp/self_tests
+    - chpl -M src -M src/quickchpl test/properties/SelfTests.chpl -o /tmp/self_tests
     - /tmp/self_tests --numTests=$QUICKCHPL_NUM_TESTS
 
 # Matrix testing for Chapel versions
 .test:matrix:
   stage: test
   script:
     - chpl --version
-    - chpl test/unit/GeneratorTests.chpl src/*.chpl -o /tmp/test
+    - chpl -M src -M src/quickchpl test/unit/GeneratorTests.chpl -o /tmp/test
     - /tmp/test
 
 test:chapel-2.6.0:
@@ -158,10 +157,10 @@ docs:chpldoc:
   stage: docs
   script:
     - mkdir -p docs/api
-    - cd src
-    - for file in *.chpl; do
+    - |
+      for file in src/quickchpl.chpl src/quickchpl/*.chpl; do
         echo "Generating docs for $file...";
-        chpldoc "$file" --output-dir ../docs/api || echo "Warning: chpldoc failed for $file";
+        chpldoc "$file" --output-dir docs/api || echo "Warning: chpldoc failed for $file";
       done
   artifacts:
     paths:

CHANGELOG.md

@@ -5,6 +5,43 @@ All notable changes to quickchpl will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/),
 and this project adheres to [Semantic Versioning](https://semver.org/).
 
+## [1.0.2] - 2026-01-15
+
+### Fixed
+- Generic type warnings: Added `Property(?)` syntax for generic formal parameters
+- Unstable symbol warnings: Replaced `_unused` loop variables with bare `for 1..n` syntax
+- Mason main module check: Restructured to proper submodule convention
+
+### Changed
+- Moved submodules to `src/quickchpl/` directory per Mason package conventions
+- Updated `quickchpl.chpl` to use `include module` statements before `public use`
+- Test files now import via `use quickchpl` instead of direct submodule imports
+- CI acceptable violations reduced from 30 to 9 (removed UnusedLoopIndex)
+- Chapel version compatibility extended to 2.8.0
+
+### Infrastructure
+- Updated GitHub Actions and GitLab CI paths for new module structure
+- Fixed Mason.toml trailing newline (Mason bug workaround)
+
+[1.0.2]: https://github.com/Jesssullivan/quickchpl/releases/tag/v1.0.2
+
+## [1.0.1] - 2026-01-12
+
+### Fixed
+- README examples updated to match actual API signatures
+- Tuple destructuring syntax corrected for Chapel 2.6+
+- Removed references to non-existent Pattern template functions
+
+### Changed
+- Consolidated `tests/` directory to `test/` (Mason convention)
+- Added chplcheck linting integration to CI
+
+### Infrastructure
+- Added GitLab CI/CD configuration
+- Fixed release workflow permissions and URLs
+
+[1.0.1]: https://github.com/Jesssullivan/quickchpl/releases/tag/v1.0.1
+
 ## [1.0.0] - 2026-01-12
 
 ### Added

Mason.lock

@@ -0,0 +1,14 @@
+
+[root]
+authors = ["Jess Sullivan <jess@sulliwood.org>"]
+chplVersion = "2.7.0..2.7.0"
+description = "A simple property-based testing library for Chapel"
+keywords = ["testing", "property-based", "quickcheck", "pbt", "fuzzing"]
+license = "MIT"
+name = "quickchpl"
+repository = "https://github.com/Jesssullivan/quickchpl"
+source = "https://github.com/Jesssullivan/quickchpl"
+type = "library"
+version = "1.0.2"
+
+

Mason.toml

@@ -1,6 +1,6 @@
 [brick]
 name = "quickchpl"
-version = "1.0.1"
+version = "1.0.2"
 type = "library"
 chplVersion = "2.6.0..2.8.0"
 license = "MIT"
@@ -15,4 +15,4 @@ keywords = ["testing", "property-based", "quickcheck", "pbt", "fuzzing"]
 
 [dev-dependencies]
 # chplcheck is required for linting (installed with Chapel)
-# Run with: chplcheck src/*.chpl test/*.chpl examples/*.chpl
+# Run with: chplcheck src/*.chpl test/*.chpl examples/*.chpl

README.md

@@ -321,11 +321,17 @@ var result = runner.check(prop);
 
 
 
+
 ### Todos & Future work:
 - [x] Integrate with Chapel Mason package repo (in progress)
 - [ ] Integrate Outbot Harness 
 - [x] Integrate with (and `[ ]` publish) sister projects, `chapel-k8s-mail`, `chapel-git`, `tinymachines`, `mariolex`)
-- [x] Add IDE and LLM friendly text and code completions (docs in the works)
+- [ ] Add IDE and LLM friendly text and code completions (docs in the works)
 - [x] Provide public demo (`aoc-2025` **done! one is good for now**)
 
+---
 
+### Notes:
+- `module include statements are not yet stable` <-- this is expected
+- `list.parSafe is unstable` <-- this is expected
+- `use of routines as values is unstable` <-- routines are necessary AFAICT to achieve this familiar PBT structure

examples/CustomGenerators.chpl

@@ -101,7 +101,7 @@ module CustomGenerators {
     }
 
     iter these(n: int = 100) ref : Color {
-      for _unused in 1..n {
+      for 1..n {
         yield this.next();
       }
     }
@@ -212,7 +212,7 @@ module CustomGenerators {
       // Show distribution (not a property test, just informative)
       writeln("\n  Color distribution (100 samples):");
       var counts: [0..7] int;
-      for _unused in 1..100 {
+      for 1..100 {
         const c = gen.next();
         select c {
           when Color.Red do counts[0] += 1;

src/quickchpl.chpl

@@ -75,13 +75,19 @@ module quickchpl {
   /* Random seed (-1 for random, or positive int for reproducibility). */
   config const seed = -1;
 
-  // Public API: Re-export core modules
+  // Public API: Include and re-export core modules
+  include module Generators;
   public use Generators;
+  include module Properties;
   public use Properties;
+  include module Shrinkers;
   public use Shrinkers;
+  include module Reporters;
   public use Reporters;
-  public use Patterns;
+  include module Combinators;
   public use Combinators;
+  include module Patterns;
+  public use Patterns;
 
   /*
     Version Information
@@ -91,7 +97,7 @@ module quickchpl {
   */
 
   /* Full semantic version string. */
-  param VERSION = "1.0.1";
+  param VERSION = "1.0.2";
 
   /* Major version number. */
   param VERSION_MAJOR = 1;
@@ -100,7 +106,7 @@ module quickchpl {
   param VERSION_MINOR = 0;
 
   /* Patch version number. */
-  param VERSION_PATCH = 1;
+  param VERSION_PATCH = 2;
 
   /*
     Quick property check with default configuration.