Update OCM setup docs

[rawagner]

Summary by CodeRabbit

• Documentation
  • Simplified development setup instructions with standardized npm commands.
  • Streamlined local integration setup process by replacing multi-step procedures with direct access guidance.
  • Enhanced developer onboarding clarity.

[coderabbitai]

📝 Walkthrough

Walkthrough

Documentation updates to DEVELOPMENT.md simplify local setup instructions by replacing multi-step processes with direct access guidance, standardizing npm commands (npm ci, npm run start), and removing uhc-portal cloning steps.

Changes

Cohort / File(s)	Summary
Documentation Simplification docs/DEVELOPMENT.md	Streamlined local development instructions by replacing npm install with npm ci, standardizing start command from npm run start:federated to npm run start, and removing multi-step uhc-portal fork/clone workflow in favor of direct URL access.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

• MGMT-20953: Add AI ChatBot #3016: Modifies docs/DEVELOPMENT.md to update yalc linking for local integration instructions, with overlapping changes to developer guidance.
• NO-ISSUE: Update DEVELOPMENT.md file with assisted-installer-app project #2912: Updates docs/DEVELOPMENT.md with concurrent edits to npm command standardization and yalc linking procedures for local Assisted Installer setup.

Suggested labels

lgtm, size/M

Suggested reviewers

• ammont82
• celdrake
• jgyselov

Poem

🐰 With nimble paws, we hop with glee,
The docs are cleaner now, you'll see!
No forking fuss, no steps so long,
Just npm ci, and we move along! 🚀

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Update OCM setup docs' directly relates to the main change, which updates documentation in docs/DEVELOPMENT.md with new setup guidance.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (1)

docs/DEVELOPMENT.md (1)

95-95: Add prerequisite link for the direct URL path.

Line 95 is a good simplification, but this URL usually depends on assisted-installer-app local setup (hosts/certs/auth context). Add a short pointer to the relevant assisted-installer-app setup docs so first-time setup doesn’t dead-end.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@docs/DEVELOPMENT.md` at line 95, Update the documentation line that shows the
direct URL "https://prod.foo.redhat.com:1337/openshift/assisted-installer" to
include a short prerequisite pointer to the assisted-installer-app local setup
docs; explicitly mention that hosts, certificates, and auth context must be
configured and add a concise link or reference text to the
assisted-installer-app setup guide (e.g., "see assisted-installer-app setup for
hosts/certs/auth") so first-time users know where to complete prerequisites
before visiting that URL.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@docs/DEVELOPMENT.md`:
- Around line 84-86: The documentation note about running `yalc link` after `npm
ci` incorrectly warns only about updating `yarn.lock`; update the text around
the `yalc link` instruction to mention `package-lock.json` (and optionally both
lockfiles) instead of or in addition to `yarn.lock`, and keep the admonition to
not commit changes to `package.json` or the relevant lockfile(s); look for the
`yalc link`/`npm ci` wording and the existing `package.json`/`yarn.lock`
references to make this wording change.

---

Nitpick comments:
In `@docs/DEVELOPMENT.md`:
- Line 95: Update the documentation line that shows the direct URL
"https://prod.foo.redhat.com:1337/openshift/assisted-installer" to include a
short prerequisite pointer to the assisted-installer-app local setup docs;
explicitly mention that hosts, certificates, and auth context must be configured
and add a concise link or reference text to the assisted-installer-app setup
guide (e.g., "see assisted-installer-app setup for hosts/certs/auth") so
first-time users know where to complete prerequisites before visiting that URL.

---

ℹ️ Review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between bdf6ed3 and 40fdf80.

📒 Files selected for processing (1)

• docs/DEVELOPMENT.md

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ElayAharoni, rawagner

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 [rawagner]

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