Skip to content

docs(issues): [#2298] add developer-tool container integration spec - #2302

Merged
josecelano merged 2 commits into
torrust:developfrom
josecelano:2298-rust-dev-tool-container-integration-spec
Sep 22, 2026
Merged

josecelano merged 2 commits into
torrust:developfrom
josecelano:2298-rust-dev-tool-container-integration-spec

Conversation

@josecelano

Copy link
Copy Markdown
Member

Summary

  • Add the folder-style specification for recurring Cargo Chef and Containerfile integration failures when adding developer-only Rust tools.
  • Define the desired invariant: developer-only tools remain absent from production tracker images and container test archives while their metadata stays available to Cargo Chef.

Scope

  • docs/issues/open/2298-rust-dev-tool-container-integration/ISSUE.md

Validation

  • linter markdown
  • linter cspell
  • contrib/dev-tools/git/hooks/pre-commit.sh
  • pre-push checks: nightly formatting, nightly workspace check, nightly documentation build, and full test suite

Related to #2298

Copilot AI lite review requested due to automatic review settings September 22, 2026 15:06
@josecelano josecelano self-assigned this Sep 22, 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.

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Copilot review overview

Review effort: Lite
Findings: 1 Medium severity

Open (1)
What changed in this PR

Adds an “issue spec” document for #2298 describing recurring failures when introducing developer-only Rust tools into the Cargo workspace while using Cargo Chef + Containerfile workflows, and defines invariants/acceptance criteria for a maintainable fix.

Changes:

  • Introduces a detailed problem model and current manual remediation checklist for dev-tool container integration failures.
  • Documents candidate solution options plus an implementation plan, verification strategy, and acceptance criteria.
  • Captures progress tracking and references tied to PR #2293 and related artifacts.
File Description
docs/​issues/​open/​2298-rust-dev-tool-container-integration/​ISSUE.md New issue-spec documenting the dev-tool/Cargo Chef/Containerfile integration problem and outlining options + acceptance criteria.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/issues/open/2298-rust-dev-tool-container-integration/ISSUE.md

@josecelano josecelano left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Verified the current PR diff against the Copilot suggestion. The reported leading || table syntax is not present, so no change was required.

Comment thread docs/issues/open/2298-rust-dev-tool-container-integration/ISSUE.md
Copilot stopped reviewing on behalf of josecelano due to an error September 22, 2026 15:32
@josecelano
josecelano force-pushed the 2298-rust-dev-tool-container-integration-spec branch from 08ad7f5 to 23c04f9 Compare September 22, 2026 15:49
@josecelano

Copy link
Copy Markdown
Member Author

ACK 23c04f9

@josecelano
josecelano merged commit 654a85d into torrust:develop Sep 22, 2026
1 check passed
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.

2 participants