Skip to content

hoist project to root and rename to ado-aw#12

Merged
jamesadevine merged 2 commits intomainfrom
refactor/hoist-project-to-root
Mar 6, 2026
Merged

hoist project to root and rename to ado-aw#12
jamesadevine merged 2 commits intomainfrom
refactor/hoist-project-to-root

Conversation

@jamesadevine
Copy link
Collaborator

No description provided.

jamesadevine and others added 2 commits March 6, 2026 22:53
- Move all source, templates, tests, and config from agentic-pipelines/ to root
- Rename package from agentic-pipelines to ado-aw (binary, log dir, all refs)
- Update GitHub Actions workflows: remove working-directory/workspaces, fix paths
- Broaden rust-pr-reviewer path triggers to match rust-tests coverage
- Update AGENTS.md architecture tree and all documentation paths

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link

github-actions bot commented Mar 6, 2026

🔍 Rust PR Review

Summary: Looks good — this is a clean, complete mechanical rename with no logic changes.

Findings

✅ What Looks Good

  • Rename is exhaustive: All occurrences of the agentic-pipelines binary/package name have been updated to ado-aw across Cargo.toml, Cargo.lock, source files, templates, workflow configs, and documentation. No stale references to the old binary name remain.
  • Log directory correctly migrated: ~/.agentic-pipelines/logs/~/.ado-aw/logs/ updated consistently in logging.rs, both templates (base.yml, 1es-base.yml), and documentation.
  • CI/CD hoisting is correct: Removing defaults.run.working-directory: agentic-pipelines and the workspaces: agentic-pipelines cache hint from daily-build.yml and rust-tests.yml is the right fix now that the Cargo workspace is at the repo root.
  • Workflow path triggers updated: on.pull_request.paths updated from agentic-pipelines/** to the explicit src/**, tests/**, templates/**, Cargo.toml, Cargo.lock — which is actually more precise than the old wildcard.
  • ADO artifact download path preserved correctly: The $(Pipeline.Workspace)/agentic-pipeline-compiler/ download target path in templates is intentionally unchanged (it's an external ADO artifact name, not this package's name), while the binary filename within it correctly changed to ado-aw.
  • No logic changes: All Rust source files are pure renames/moves with only string-literal and comment updates — no behavioral changes introduced.

Generated by Rust PR Reviewer for issue #12 ·

@jamesadevine jamesadevine merged commit 9c10cf2 into main Mar 6, 2026
6 checks passed
@jamesadevine jamesadevine deleted the refactor/hoist-project-to-root branch March 6, 2026 23:00
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