Skip to content

Feature: Manifest Validation#57

Merged
bakaq merged 14 commits intobakaq:mainfrom
constraintAutomaton:validation
Oct 24, 2025
Merged

Feature: Manifest Validation#57
bakaq merged 14 commits intobakaq:mainfrom
constraintAutomaton:validation

Conversation

@constraintAutomaton
Copy link

@constraintAutomaton constraintAutomaton commented Oct 18, 2025

Feature: Manifest Validation

This PR introduces manifest validation that runs before package installation. Additionally, all packages are now required to specify a license.

Changes

  • Added pre-installation manifest validation
  • Enforced license requirement for all packages

Future Work

  • Add CLI flag to validate manifests without performing installation (e.g., --validate-only)
  • Better handling of multiple licenses

closes #23

Copy link
Owner

@bakaq bakaq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. I've pointed out some places that can improve in terms of purity. Validation of the manifest is something that could certainly be done in just pure and monotonic Prolog, though it's certainly very annoying without something like Qupak.

I also really like the enforcement of licenses.

Copy link
Owner

@bakaq bakaq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice integration of Qupak! There is still some cleaning up of the module-related stuff left though.

I've put some background on operators in Qupak and why we might still want to have them, and some suggestions where the code can be cleaned up a bit.

Copy link
Owner

@bakaq bakaq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Let's get this in!

@bakaq bakaq merged commit 56ad97d into bakaq:main Oct 24, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Manifest validation

3 participants