Skip to content

Conversation

@prabhu
Copy link
Contributor

@prabhu prabhu commented Aug 23, 2025

CPAN namespace rules are too difficult to represent in JSON Schema format. Gemini and I tried our best to convey the information from the extended definition in the current format.

  • Defined Requirements: The name_definition is now correctly marked as "required", while the version_definition is marked as "optional". The requirement for all keys in the qualifiers_definition has also been explicitly set to "optional".
  • Improved Namespace and Name Definitions: The namespace_definition now specifies its native_name as "author id" and includes a normalization_rules field to enforce that it must be uppercased. The note in name_definition has been updated to be more explicit about the different validation rules that apply depending on whether a namespace is present (for a distribution) or absent (for a module).
  • Added Case Sensitivity: The namespace_definition now correctly includes case_sensitive: true.

Signed-off-by: Prabhu Subramanian <prabhu@appthreat.com>
@giterlizzi
Copy link
Contributor

Hi, this CPAN specification will be replaced by this PR #420.

@prabhu
Copy link
Contributor Author

prabhu commented Aug 24, 2025

Let's keep this open. Once 420 is merged will rebase and redo to update the models

@prabhu prabhu marked this pull request as draft August 24, 2025 20:15
@sjn
Copy link
Contributor

sjn commented Sep 12, 2025

Seems I made a mistake in the merge; Apologies!

sjn added 2 commits September 14, 2025 04:03
Add a clarification that on CPAN, packages are called "distributions"
Re-introduce prabhu's changes (apologies for deleting them)
@sjn sjn requested review from pombredanne and sjn September 14, 2025 02:10
Copy link
Contributor

@sjn sjn left a comment

Choose a reason for hiding this comment

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

LGTM

@prabhu prabhu marked this pull request as ready for review September 15, 2025 12:33
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.

4 participants