Add JSON schema to manifest so it is installed (#1017)

Author: jaimergp

.github/workflows/main.yml

@@ -93,6 +93,7 @@ jobs:
           activate-environment: constructor-dev
           environment-file: dev/environment.yml
           python-version: ${{ matrix.python-version }}
+          miniconda-version: latest
       - name: Install AzureSignTool
         if: startswith(matrix.os, 'windows')
         run: dotnet.exe tool install --global AzureSignTool
@@ -196,18 +197,13 @@ jobs:
   build:
     name: Canary Build
     needs: [tests]
-    # only build canary build if
-    # only build canary build if
+    # only upload canary build if
     # - prior steps succeeded,
     # - this is the main repo, and
     # - we are on the main (or feature) branch
     if: >-
       success()
       && !github.event.repository.fork
-      && (
-        github.ref_name == 'main'
-        || startsWith(github.ref_name, 'feature/')
-      )
     strategy:
       matrix:
         include:
@@ -228,8 +224,16 @@ jobs:
           ref: ${{ github.ref }}
           clean: true
           fetch-depth: 0
+
+      - uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065 # v5.6.0
+      - name: Build Python sdist and wheel
+        run: |
+          python -m pip install build
+          python -m build --sdist --wheel . --outdir dist
+
       - name: Create and upload canary build
         uses: conda/actions/canary-release@eb545bb8ab48d499b31c057a6df3cf46753fdbcb # v25.3.1
+        if: github.ref_name == 'main' || startsWith(github.ref_name, 'feature/')
         with:
           package-name: ${{ github.event.repository.name }}
           subdir: ${{ matrix.subdir }}

MANIFEST.in

@@ -1,2 +1,2 @@
-include versioneer.py
 include constructor/_version.py
+include constructor/data/construct.schema.json

constructor/fcp.py

@@ -443,7 +443,7 @@ def _main(
             input_dir=input_dir,
         )
     if dry_run:
-        return None, None, None, None, None, None, None
+        return None, None, None, None, None, None, None, None
     pc_recs, _urls, dists, has_conda = _fetch_precs(
         precs, download_dir, transmute_file_type=transmute_file_type
     )

constructor/main.py

@@ -369,7 +369,7 @@ def _build_message(self):
         )
 
 
-def main():
+def main(argv=None):
     logging.basicConfig(level=logging.INFO)
     p = argparse.ArgumentParser(description="build an installer from <DIRECTORY>/construct.yaml")
 
@@ -448,7 +448,7 @@ def main():
         metavar="DIRECTORY",
     )
 
-    args = p.parse_args()
+    args = p.parse_args(argv)
     logger.info("Got the following cli arguments: '%s'", args)
 
     if args.verbose or args.debug:

news/740-manifest-data

@@ -0,0 +1,20 @@
+### Enhancements
+
+* <news item>
+
+### Bug fixes
+
+* Include `construct.schema.json` file as part of the source distribution. (#740)
+* Fix return value when running `constructor --dry-run` mode. (#740)
+
+### Deprecations
+
+* <news item>
+
+### Docs
+
+* <news item>
+
+### Other
+
+* <news item>

recipe/meta.yaml

@@ -47,6 +47,8 @@ test:
     - constructor --help
     # Run unit tests
     - pytest -v tests -k "not examples"
+    # Run _one_ example as a smoke integration test
+    - pytest -v tests/test_examples.py::test_example_miniforge
 
 about:
   home: https://conda.io

tests/test_main.py

@@ -0,0 +1,19 @@
+from textwrap import dedent
+
+from constructor.main import main
+
+
+def test_dry_run(tmp_path):
+    inputfile = dedent(
+        """
+        name: test_schema_validation
+        version: X
+        installer_type: all
+        channels:
+          - http://repo.anaconda.com/pkgs/main/
+        specs:
+          - ca-certificates
+        """
+    )
+    (tmp_path / "construct.yaml").write_text(inputfile)
+    main([str(tmp_path), "--dry-run"])