Skip to content

new module properties#215

Open
raphael-goetz wants to merge 2 commits intomainfrom
#212-new-module-properties
Open

new module properties#215
raphael-goetz wants to merge 2 commits intomainfrom
#212-new-module-properties

Conversation

@raphael-goetz
Copy link
Copy Markdown
Member

Resolves: #212

Copilot AI review requested due to automatic review settings April 28, 2026 15:35
@github-actions
Copy link
Copy Markdown

GitLab Pipeline Action

General information

Link to pipeline: https://gitlab.com/code0-tech/development/draco/-/pipelines/2485892542

Status: Passed
Duration: 2 minutes

Job summaries

docs:preview

Documentation preview available at https://code0-tech.gitlab.io/-/development/telescopium/-/jobs/14126374275/artifacts/out/index.html

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates Draco’s runtime status reporting to align with the newer tucana API for “module properties” by switching from RuntimeFeature/AdapterConfiguration to AdapterStatusConfiguration, and bumps workspace dependencies accordingly (Issue #212).

Changes:

  • Refactors ServerRunner::serve and DracoRuntimeStatusService to send AdapterStatusConfiguration only (removing runtime features).
  • Updates REST and CRON adapters to use the new serve(configs) signature and build the new configuration payload.
  • Bumps code0-flow to 0.0.33 and tucana to 0.0.70.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
crates/base/src/runner.rs Changes serve API to accept only AdapterStatusConfiguration and forwards it into runtime status reporting.
crates/base/src/client/mod.rs Adjusts runtime status update payload to include only configurations (new config type).
adapter/rest/src/main.rs Builds and reports REST endpoint configuration via AdapterStatusConfiguration.
adapter/cron/src/main.rs Updates call-site for new serve signature (currently passes empty config list).
Cargo.toml Dependency bumps for code0-flow and tucana.
Cargo.lock Lockfile updates to reflect dependency bumps.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread adapter/cron/src/main.rs
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.

Draco should include the modules properties

2 participants