Skip to content

Engine integration#3

Merged
6igby merged 67 commits intomainfrom
engine-integration
Sep 4, 2025
Merged

Engine integration#3
6igby merged 67 commits intomainfrom
engine-integration

Conversation

@6igby
Copy link
Owner

@6igby 6igby commented Sep 4, 2025

integrating engine into main

Aditya Hindocha and others added 30 commits July 30, 2025 01:24
…aluation

This pull request enhances the main.py script in the compliance engine project with several key improvements:
	•	Added error handling
	•	Handles missing or unreadable configuration and rule files
	•	Catches and reports invalid JSON formats gracefully
	•	Implemented rule validation
	•	Verifies that each rule JSON includes the required fields: id, tag, expected_setting, path, and description
	•	Skips invalid rules while logging a warning
	•	Replaced hardcoded tag mapping
	•	Introduced get_value_by_path() function to dynamically evaluate configuration values based on rule-defined paths
	•	Allows easier scalability for additional and more complex rule structures
	•	Improved output and reporting
	•	Clearly logs PASS/FAIL status for each rule along with the reason for failures
	•	Summarizes total number of rules evaluated, with a count of passed and failed rules
	•	Enhanced code readability
	•	Added inline comments for clarity and maintainability
	•	Organized logic to support easier team collaboration and future development
@6igby
Copy link
Owner Author

6igby commented Sep 4, 2025

integrating monorepo merge. all files from Compliance-Engine branch in https://github.com/223159756/AutoAudit_tester/tree/engine-merge moved into main of this fork in /engine directory.

@6igby 6igby merged commit cd1e4d8 into main Sep 4, 2025
31 of 33 checks passed
@6igby 6igby deleted the engine-integration branch September 4, 2025 05:49
6igby added a commit that referenced this pull request Sep 4, 2025
6igby added a commit that referenced this pull request Sep 11, 2025
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