OSAC-3818: add bare-metal-fulfillment-operator and osac-csi-driver to dependency ordering

[eranco74]

Summary

• Adds bare-metal-fulfillment-operator and osac-csi-driver to the cross-component dependency ordering in AGENTS.md
• Replaces the linear chain with a DAG diagram showing the actual dependency structure (bare-metal-fulfillment-operator depends on osac-operator, osac-csi-driver depends on fulfillment-service)

Test plan

• Verify the DAG accurately reflects Go module dependencies
• Confirm all mono-repo components are now represented

🤖 Generated with Claude Code

Summary by CodeRabbit

• Documentation
  • Expanded cross-component change guidance with a dependency diagram covering parallel dependencies, including bare-metal and CSI components.
  • Retained the installer as the final dependency in the documented sequence.
  • Preserved links to deployment coordination guidance.

[openshift-ci-robot]

@eranco74: This pull request references OSAC-3818 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target the "5.0.0" version, but no target version was set.

Details

In response to this:

Summary

• Adds bare-metal-fulfillment-operator and osac-csi-driver to the cross-component dependency ordering in AGENTS.md
• Replaces the linear chain with a DAG diagram showing the actual dependency structure (bare-metal-fulfillment-operator depends on osac-operator, osac-csi-driver depends on fulfillment-service)

Test plan

• Verify the DAG accurately reflects Go module dependencies
• Confirm all mono-repo components are now represented

🤖 Generated with Claude Code

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: eranco74

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [eranco74]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository: osac-project/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 707ad690-a926-44f1-9b43-f543f5ee7015

📥 Commits

Reviewing files that changed from the base of the PR and between bd04c8c and f7976a2.

📒 Files selected for processing (1)

• AGENTS.md

🚧 Files skipped from review as they are similar to previous changes (1)

• AGENTS.md

---

Walkthrough

AGENTS.md now uses a dependency diagram for cross-component guidance. The diagram includes bare-metal and CSI components and keeps osac-installer dependent on all preceding components. Deployment coordination guidance remains linked to osac-installer/AGENTS.md.

Changes

Cross-component dependency guidance

Layer / File(s)	Summary
Dependency diagram AGENTS.md	Replaces the inline dependency description with a diagram covering the fulfillment service, operators, CSI driver, bare-metal operator, and installer.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

• osac-project/osac#104: Updates the root AGENTS.md dependency guidance.
• osac-project/osac#213: Adds osac-csi-driver documentation related to the dependency guidance.
• osac-project/osac#218: Updates cross-component dependency documentation with osac-metering.

Suggested labels: approved, jira/valid-reference

Suggested reviewers: amito, jhernand

🚥 Pre-merge checks | ✅ 11

✅ Passed checks (11 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly identifies the two components added to the dependency ordering, which matches the main change.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
No-Hardcoded-Secrets	✅ Passed	The PR changes only AGENTS.md; all added lines are dependency documentation, and the credential-pattern scan found no secrets, credential URLs, private keys, or encoded blobs.
No-Weak-Crypto	✅ Passed	The PR-specific diff changes only AGENTS.md; added lines contain no weak algorithms, crypto APIs, custom crypto, or secret comparisons.
No-Injection-Vectors	✅ Passed	The PR changes only AGENTS.md; its added dependency diagram and guidance contain no SQL concatenation, shell=True, eval/exec, unsafe loading, os.system, or dangerouslySetInnerHTML.
Container-Privileges	✅ Passed	The PR changes only AGENTS.md. The exact patch adds no Kubernetes/container manifest and no prohibited privilege setting.
No-Sensitive-Data-In-Logs	✅ Passed	The PR-side merge comparison changes only AGENTS.md; it adds dependency documentation and contains no logging code or sensitive-data output.
Ai-Attribution	✅ Passed	AI use is disclosed in the PR and commit; the PR commit has Assisted-by: Claude Code <noreply@anthropic.com> and no AI Co-Authored-By trailer.

✨ Finishing Touches 💡 1

🛠️ Fix failing CI checks 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Note

GitHub couldn't provide a complete incremental comparison for this pull request, so CodeRabbit is performing a full review instead. This review may take a little longer.

[omer-vishlitzky]

/lgtm