This repository was archived by the owner on Feb 4, 2026. It is now read-only.
Merged
Conversation
- Add deployment timing metrics (full, net, per-transition) - Add DeploymentMetricsCollector for local/remote metrics aggregation - Add DeploymentMetricsScheduler for leader-based distribution - Use MessageRouter for deployment event dispatch - Add topology change handling for stale metrics cleanup - Add run scripts for CLI, Node, and Forge tools - Update CLI reference and getting-started documentation 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Replace awaitility with simple polling utility - Update CI workflow to trigger on release branches 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add 'aether blueprint apply' command for TOML blueprints - Add /blueprint POST endpoint to ManagementServer - Rename CLI from aether-cli to aether - Rename script from aether-cli.sh to aether.sh - Update documentation with new commands 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add DHTConfig with full replication support (0 = all nodes store everything) - Wire DHT, ArtifactStore, and MavenProtocolHandler into AetherNode - Add /repository/** routes to ManagementServer for production use - Add /repository/** routes to ForgeApiHandler for testing - Update dependency management for artifact-repo module 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- CLI outputs aether.jar (renamed from aether-cli.jar) - Node outputs aether-node.jar with shade plugin - Forge outputs aether-forge.jar - Add install.sh for quick installation - Add release.yml workflow for GitHub releases on tag push - Update README with installation instructions 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughVersion bump from 0.6.1 to 0.6.2 introducing deployment metrics infrastructure for tracking slice deployment timing, artifact repository integration via DHT and Maven protocol handler, blueprint support for slice configuration, automated CI/release workflows, and installation scripts. Removes slice-annotations module and related classloader infrastructure. Changes
Sequence Diagram(s)sequenceDiagram
actor Node1
actor Node2
participant DMC as DeploymentMetricsCollector
participant DMS as DeploymentMetricsScheduler
participant Net as ClusterNetwork
actor Leader
Note over Leader,Node2: Deployment Metrics Collection & Broadcasting
Node1->>DMC: onDeploymentStarted(event)
Note over DMC: Track in-progress deployment
Node1->>DMC: onStateTransition(LOAD→LOADED)
Note over DMC: Update load timing
Node1->>DMC: onDeploymentCompleted(event)
Note over DMC: Finalize, move to completed<br/>with retention policy
rect rgb(200, 220, 255)
Note over Leader: Leader periodically broadcasts (5s interval)
Leader->>DMS: Triggered by interval
DMS->>DMC: collectLocalEntries()
DMC-->>DMS: Return metrics map
DMS->>Net: send(DeploymentMetricsPing)
end
Net->>Node1: onDeploymentMetricsPing(ping)
Note over Node1: If sender != self
Node1->>Net: send(DeploymentMetricsPong)
Net->>Leader: onDeploymentMetricsPong(pong)
Leader->>DMC: onDeploymentMetricsPong(pong)
Note over DMC: Merge remote metrics<br/>Enforce retention per artifact
rect rgb(240, 200, 200)
Note over Node1: On topology change
DMC->>DMC: onTopologyChange(nodeRemoved)
Note over DMC: Cleanup in-progress &<br/>remote metrics for node
end
Note over DMC: Query interface<br/>allDeploymentMetrics()<br/>metricsFor(artifact)
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Possibly related PRs
Poem
✨ Finishing touches
📜 Recent review detailsConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (53)
Comment |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
aether)install.shChanges
🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Documentation
Chores
✏️ Tip: You can customize this high-level summary in your review settings.