Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Code refactoring and cleanup for 1.0.0 release #204

Merged
merged 42 commits into from
Sep 26, 2023
Merged

Code refactoring and cleanup for 1.0.0 release #204

merged 42 commits into from
Sep 26, 2023

Conversation

david-waltermire
Copy link
Collaborator

@david-waltermire david-waltermire commented Aug 2, 2023 •
edited
Loading

Committer Notes

This PR contains refactored code to prepare for a 1.0 codebase.

The following refactoring work is included.

  • Combine metaschema-model-common and metaschema-model into a core module.
    • Combine modules
    • Refactor package names to use gov.nist.secauto.metaschema.core
    • Create relocation pom.xml entries
  • Combine metaschema-java-codegen and metaschema-java-binding into a databind module.
    • Merge modules
    • Refactor package names to use gov.nist.secauto.metaschema.databind
    • Create relocation pom.xml entries
  • Migrate content parsing code out of the model classes into format specific parser classes. Reduce code where possible.
    • XML
    • JSON/YAML
  • Migrate content generation code out of the model classes into format specific parser classes. Reduce code where possible.
    • XML
    • JSON/YAML
  • Fully implement problem handlers in parsing code (relates to CLI Silently Drops Undefined Assembly Instances During Format Conversion #131)
    • XML
    • JSON/YAML
  • Properly handle resource closure in content parsing and generating code
    • Parsing
      • XML
      • JSON/YAML
    • Generation
      • XML
      • JSON/YAML
  • Metapath improvements
    • Refactor Metapath focus to allow for Metapath evaluations against no focus, as well as cases where the focus is not a node item.
    • Improved memory footprint for Axis expressions to use static objects for evaluation.
    • Refactor the Step and ParentItem ASTs to use the new Axis support.
  • Implement JPMS module-info.java declarations for modules. This will help to better identify and separate the stable API from implementation classes.
    • core
    • databind
    • metaschema-schema-generator

Many of these changes are breaking changes for users of the API, which is why these changes are targeting a major release.

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you squashed any non-relevant commits and commit messages? [instructions]
  • Do all automated CI/CD checks pass?

Changes to Core Features:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your core changes, as applicable?
  • Have you included examples of how to use your new feature(s)?
  • Have you updated all website](https://pages.nist.gov/metaschema) and readme documentation affected by the changes you made? Changes to the website can be made in the website/content directory of your branch.

@david-waltermire david-waltermire force-pushed the feature-reorganize-modules-for-1.0 branch from c3d6207 to 6caba4f Compare August 4, 2023 00:54
This was referenced Aug 4, 2023
Open
Open
@david-waltermire david-waltermire force-pushed the feature-reorganize-modules-for-1.0 branch from 62355f3 to 6eccab1 Compare August 14, 2023 22:03
This was linked to issues Aug 14, 2023
CLI Silently Drops Undefined Assembly Instances During Format Conversion #131
Closed
Reduce the number of submodules to form a core API #195
Closed
@david-waltermire david-waltermire force-pushed the feature-reorganize-modules-for-1.0 branch from 6eccab1 to 98c0b29 Compare August 15, 2023 22:23
@david-waltermire david-waltermire force-pushed the feature-reorganize-modules-for-1.0 branch 3 times, most recently from dd454d0 to 8c9f969 Compare August 28, 2023 02:59
@david-waltermire david-waltermire marked this pull request as ready for review August 28, 2023 23:40
@david-waltermire david-waltermire force-pushed the feature-reorganize-modules-for-1.0 branch from eaaf338 to 0628bf3 Compare August 28, 2023 23:42
@david-waltermire david-waltermire force-pushed the feature-reorganize-modules-for-1.0 branch from d2b5e22 to 0c4f8ac Compare September 20, 2023 15:53
@david-waltermire david-waltermire force-pushed the feature-reorganize-modules-for-1.0 branch from 0c4f8ac to ebdc62e Compare September 21, 2023 12:45
@david-waltermire
Added metaschema as core submodule.
742caa2 
Relocated metaschema-model-common to metaschema-core.
Relocated metaschema-model to metaschema-core.
@david-waltermire
Cleaned up reports.
7a47949
@david-waltermire
Ensure generated resources are included for site generation.
64d0d06
@david-waltermire
Added true() and false() metapath function.
f6046ad 
Refactored error handling for step-related Metapath expressions to ensure the provided context node item is non-null and a document. This also lays ground work for null node contexts in Metapath expressions that do not require a focused node.
@david-waltermire
Refactored code to remove the IRequiredValue* interfaces.
a4542dd
@david-waltermire
Refactored INodeItem handling to implement the following correct beha…
bf4fe22 
…viors:

- IDocumentNodeItem instances will never have a value (i.e. getValue == null)
- Root metapaths (i.e. `/`) must be executed against an IDocumentNodeItem or an error should be raised.
@david-waltermire
Cleaned up some unneeded generics in the implementation.
662bd64
@david-waltermire
Refactored node item implementation to provide better interface segre…
54ef204 
…gation and to limit class responsibility.
@david-waltermire
Migrated node item classes to a new package.
27f7a6c 
Added some Javadoc comments.
@david-waltermire
Relocated unit tests to match new package structure.
c99651e 
Added a new mock model builder (MockedModelTestSupport) to facilitate creation of unit tests around using a mocked Metaschema model.
@david-waltermire
Refactored Metapath focus to allow for Metapath evaluations against n…
25568a5 
…o focus, as well as cases where the focus is not a node item.

Improved memory footprint for Axis expressions, which now use static objects for evaluation. Refactored the Step and ParentItem ASTs to use the new Axis support.
@david-waltermire
Relocated the "metaschema-java-binding" and "metaschema-java-codegen"…
b0442eb 
… modules to the "databind" module.
@david-waltermire
Refactored XML parsing, extracting XML parsing logic into the XmlPars…
58fe02d 
…er class. This resulted in a cleaner set of bound definition/property classes. Also optimized deserialization to return the Object value directly, instead of first creating an unneeded node item.
@david-waltermire
Removed aggregate javadoc.
33311c4
@david-waltermire
Added support for function namespaces.
9208aae 
Some Javadoc improvements.
Refactored code generation, moving code generation methods into a dedicated class, DefaultMetaschemaClassFactory.java.
Refactored code generation production classes to simplify and reduce the number of classes.
@david-waltermire
Implemented XML problem handler for parsing. Further code cleanup and…
51d255b 
… refactoring of XML parsing code.

Created more javadocs.
Completed some code reformatting.
@david-waltermire
Implemented the JSON problem handler. Resolved usnistgov#131.
af2f53d 
Refactored JSON parser tests.
Eliminated a significant number of compile, PMD, and Spotbugs warnings.
@david-waltermire
Refactored input stream handling. Input streams should now be properl…
20c36be 
…y closed.
@david-waltermire
Work towards reorganizing the Metaschema model interfaces and impleme…
f63ba81 
…ntations.
@david-waltermire
Updated core/metaschema submodule.
abbb570
@david-waltermire
Refactored XML and JSON generation by moving generation code into for…
00ffd43 
…mat specific classes (i.e. MetaschemaJsonWriter, MetaschemaXmlWriter).

Adjusted problem handlers to use interface defaults and a common implementation of default handling for missing instances.
Renamed MetaschemaXmlParser to MetaschemaXmlReader, MetaschemaJsonParser to MetaschemaJsonReader, and MetaschemaXmlGenerator to MetaschemaXmlWriter.
@david-waltermire
Moved property info factory to static method on IModelPropertyInfo.
34a2203 
Adjusted IDataTypeHandler creation to avoid extra method callbacks.
Reduced unnecessary overridden methods.
@david-waltermire
Removed the need for specific root assembly model interfaces and clas…
5c15f6f 
…ses.
@david-waltermire
Removed the remaining CPD duplications in core.
8339474 
Cleaned up unneed classes and methods.
Reduced PMD warnings.
Created Javadocs.
@david-waltermire
Refactored JSON reader and writer to use common instance logic. The c…
bccb333 
…ollection of instances are now lazy generated and cached, improving read and write performance for commonly accessed Metaschema-based JSON objects.
@david-waltermire
Cleaned up some unused/unneeded classes and methods.
755f5f6 
Refactored binding context to allow for dynamic class generation, compilation, and loading. Removed DynamicBindingContext.java, which is no longer needed.
Cleaned up unused dependencies.
Added Javadoc comments.
@david-waltermire
Added a metaschema-databind module-info module descriptor.
1390765
@david-waltermire
Removed metaschema-model/metaschema submodule.
13e04af 
Moved metaschema-schema-generator to schemagen.
@david-waltermire
Renamed IMetaschema to IModule to be more consistent with terminology.
1726cbd 
Deleted unused submodule.
@david-waltermire
Added JPMS module descriptor for the schemagen module. Addresses reso…
04bed9f 
…urce loading and compilation issues due to module restrictions.

Generate test classes in a temp directory.
@david-waltermire
Removed dependency on JAXB.
3415c70
@david-waltermire
Enhanced index key miss error reporting to include the key data that …
00efb30 
…missed.
@david-waltermire
Adjusted YAML parser configuration to set a higher default codepoint …
db5980b 
…limit.
@david-waltermire
Adjusted generate sources mojo default phase to a more reasonable val…
ac06907 
…ue (generate-sources).
@david-waltermire
Added new String initializer for IAnyAtomicItem.
8adb754
@david-waltermire
Improved some logging messages.
67aa962
@david-waltermire
Improved XML schema generation testing.
04cfcc0
@david-waltermire
Fixed compile error in XML schema generation test.
b363b00
@david-waltermire
Use treeless Git clones on GHA builds
bc1d0b6
@david-waltermire david-waltermire force-pushed the feature-reorganize-modules-for-1.0 branch from 0489712 to bc1d0b6 Compare September 26, 2023 16:49
@david-waltermire david-waltermire merged commit 2e91fac into usnistgov:develop Sep 26, 2023
@david-waltermire david-waltermire deleted the feature-reorganize-modules-for-1.0 branch September 26, 2023 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@david-waltermire