Skip to content

Feature/cicd#87

Merged
SickleFire merged 3 commits into
SickleFire:devfrom
Samanyu-dev:feature/cicd
Jul 4, 2026
Merged

Feature/cicd#87
SickleFire merged 3 commits into
SickleFire:devfrom
Samanyu-dev:feature/cicd

Conversation

@Samanyu-dev

Copy link
Copy Markdown
Collaborator

This PR adds the --diff-only argument to the CI export capabilities, which filters the exported heap blocks to include only those that are new or changed relative to the baseline snapshot taken at startup. This ensures that massive JSON/CSV files don't become bogged down with identical unmodified blocks.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Chore / Refactor

Related issues

How Has This Been Tested

  • Unit tests: cargo test --lib
  • Integration tests: cargo test --test integration_tests
  • Manual steps:
    • Executed mvis ci to verify the --diff-only validation rejects executions missing --leak-check.
    • Exported a differential heap JSON and observed significantly smaller payload sizes.
  • Platform tested on: macOS

Checklist

  • I have read the CONTRIBUTING.md and followed the repo guidelines.
  • Code builds locally (cargo build --release)
  • All tests pass (cargo test)
  • New and existing tests cover my changes
  • I added/updated documentation where applicable
  • I updated CHANGELOG or release notes if needed

CI and Performance Notes

  • Performance impact: Significantly reduces file I/O operations and final artifact sizes when creating heap snapshots of large processes by filtering unmodified baseline entries prior to serialization.

This commit adds the --diff-only argument to the CI export capabilities, which filters the exported heap blocks to include only those that are new or changed relative to the baseline snapshot taken at startup. This ensures that massive JSON/CSV files don't become bogged down with identical unmodified blocks.
@Samanyu-dev Samanyu-dev requested a review from SickleFire July 4, 2026 12:10
@Samanyu-dev Samanyu-dev self-assigned this Jul 4, 2026
Copilot AI review requested due to automatic review settings July 4, 2026 12:10
@Samanyu-dev Samanyu-dev added the enhancement New feature or request label Jul 4, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@SickleFire SickleFire left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM! consistent formatting and styling like with the other flags.

@SickleFire SickleFire merged commit 0cf5fb9 into SickleFire:dev Jul 4, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants