Skip to content

Feature/artifact blocklist publish audit#10

Closed
Techikrish wants to merge 3 commits into
mainfrom
feature/artifact-blocklist-publish-audit
Closed

Feature/artifact blocklist publish audit#10
Techikrish wants to merge 3 commits into
mainfrom
feature/artifact-blocklist-publish-audit

Conversation

@Techikrish

Copy link
Copy Markdown
Owner

Description

Related Issue

Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • 📚 Documentation update
  • 🔧 Chore (CI/CD, dependencies, configuration)

Testing

  • Tests added/updated
  • All 92 tests passing locally
  • Code coverage > 80%

Test command:

pytest tests/ -v

Checklist

  • My code follows the PEP 8 style guide
  • I have performed a self-review of my own code
  • I have added tests for new functionality
  • All new and existing tests pass
  • I have updated documentation (if needed)
  • My changes do not introduce breaking changes
  • PR targets main branch

Additional Context

- Feature 1: Artifact Pattern Blocklist
  * Detects dangerous file patterns before commit
  * Supports sourcemaps, build artifacts, private keys, env files, IDE metadata
  * Configurable via .shieldcommit.yaml with severity levels
  * Blocks commits with error-level violations

- Feature 2: Publish Audit (audit-publish command)
  * Pre-flight checks before publishing to npm or PyPI
  * Simulates package content and scans for violations
  * Supports table and JSON output formats
  * Auto-detects package type (npm/pypi)
  * Exit codes: 0=clean, 1=warnings, 2=errors

- New modules:
  * config.py: Configuration parser for .shieldcommit.yaml
  * artifact_detector.py: Artifact pattern detection logic
  * publish_audit.py: Publish audit implementation

- Updated:
  * scanner.py: Integrated artifact detection
  * __main__.py: Added audit-publish command, updated scan output
  * requirements.txt: Added pyyaml, toml dependencies

- Documentation:
  * FEATURES.md: Comprehensive feature documentation
  * .shieldcommit.yaml: Example configuration file with all options

- Tests:
  * test_artifact_detector.py: Tests for artifact patterns
  * test_config.py: Tests for configuration loading
@Techikrish Techikrish closed this Apr 12, 2026
@Techikrish

Copy link
Copy Markdown
Owner Author

added feature has some bugs and error

@Techikrish
Techikrish deleted the feature/artifact-blocklist-publish-audit branch April 12, 2026 13:11
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.

1 participant