Repository navigation
docs: move README guidance to the official documentation website - #1406
Merged
Wassim Chegham (manekinekko) merged 6 commits intoOct 5, 2026
Merged
Conversation
Wassim Chegham (manekinekko)
marked this pull request as ready for review
September 18, 2026 07:51
Wassim Chegham (manekinekko)
requested review from
Cedric Vidal (cedricvidal)
and
a balanced review from Copilot
September 18, 2026 07:51
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
Several promoted documentation paths retain obsolete repository references, and the first-run landing description is inaccurate.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Moves detailed README guidance into the official documentation website.
Changes:
- Adds local development, contributor, development, support, and security guides.
- Expands platform concepts and evaluation guidance.
- Updates navigation and repository entry points.
File summaries
| File | Description |
|---|---|
README.md |
Replaces detailed guidance with website links. |
CONTRIBUTING.md |
Updates documentation contribution guidance. |
website/README.md |
Documents the website’s new role. |
website/astro.config.mjs |
Registers new sidebar pages. |
website/src/content/docs/index.mdx |
Adds setup and contribution cards. |
website/src/content/docs/getting-started/access.md |
Links local setup guidance. |
website/src/content/docs/getting-started/first-run.md |
Updates first-run prerequisites. |
website/src/content/docs/getting-started/local-development.md |
Adds comprehensive local setup instructions. |
website/src/content/docs/introduction/concepts.md |
Adds evaluation building blocks. |
website/src/content/docs/introduction/what-is-scope.md |
Expands architecture and interpretation guidance. |
website/src/content/docs/resources/contributing.md |
Adds contribution and licensing guidance. |
website/src/content/docs/resources/development.md |
Adds repository and development references. |
website/src/content/docs/resources/support.md |
Adds support, security, and responsible-use guidance. |
Review details
- Files reviewed: 13/13 changed files
- Comments generated: 3
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Test Results (Node.js 22)test: Run #70
🎉 All tests passed! |
Test Results (Node.js 22)test: Run #71
🎉 All tests passed! |
Test Results (Node.js 22)test: Run #72
🎉 All tests passed! |
Josh Duffney (duffney)
requested review from
Josh Duffney (duffney)
and removed request for
Cedric Vidal (cedricvidal)
September 21, 2026 17:22
Wassim Chegham (manekinekko)
force-pushed
the
manekinekko-documentation-migration
branch
from
September 24, 2026 14:57
564f84f to
10d841f
Compare
Test Results (Node.js 22)test: Run #93
🎉 All tests passed! |
Keep the README focused on introducing Scope and directing users and contributors to the official documentation. Move setup, development, contribution, support, and licensing details into navigable website pages. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Align installation docs, release discovery, self-update downloads, and the publisher with the current repository. Build shared dependencies before publishing and cover installer transports and release lookups with regression tests. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Use this microsoft/scope checkout as the documentation source of truth. Verify source-map links and clarify website, OpenAPI, and Pages workflow paths. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Wassim Chegham (manekinekko)
force-pushed
the
manekinekko-documentation-migration
branch
from
September 24, 2026 15:28
10d841f to
1128ffe
Compare
Test Results (Node.js 22)test: Run #96
🎉 All tests passed! |
Resolve conflicts with the public CLI release work (#1442) and the website redesign (#1400): - Adopt main's canonical root installer, publish workflow, update command, and CLI distribution docs; drop the superseded website installer test while keeping the extra release-lookup test cases. - Keep the slim README and port main's prompt-evaluation additions to the website development guide. - Keep main's Home.astro landing and link local development, contributing, and support from it. - Point local CLI usage at an explicit SCOPE_API_URL now that the CLI has no default API URL, and link the community review policy. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Test Results (Node.js 22)test: Run #166
🎉 All tests passed! |
6 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Summary
apps/cli/src/utils/update-check.test.ts: a customSCOPE_RELEASES_URLbypassesgh, and the lookup returns no version when nocli/v*release exists.Merge with
mainThis branch originally also migrated the CLI installer, release workflow, and self-update to
microsoft/scope. #1442 has since landed the canonical public installer and release workflow, so those changes now usemain's implementation:.github/workflows/publish-cli.yml,apps/cli/README.md,apps/cli/src/commands/update.ts,website/install-cli.sh, anddocs/architecture/cli-distribution.mdmatchmain. The supersededwebsite/install-cli.test.mjswas removed;scripts/install-cli.test.tsonmaincovers the installer.curlinstaller frommainand still point to Local development when no CLI release is available.Home.astro, so the homepage now has a row of links to Local development, Contributing, and Support & security instead of two new cards.main(Add static prompt quality evaluations #1441) moved into the website development guide.SCOPE_API_URLexplicitly because the CLI no longer has a default API URL.CONTRIBUTING.md(docs: define community contribution review and merge policy #1413).Demo
N/A: documentation changes and unit tests only; no Portal or CLI behavior changes.
Testing
After merging
main:pnpm exec vitest run apps/cli/src/utils/update-check.test.ts apps/cli/src/commands/update.test.ts scripts/install-cli.test.ts: 25 tests passed.cd website && pnpm test: all 7 plugin tests passed.cd website && SITE=https://microsoft.github.io BASE_PATH=/scope pnpm build: passed with the public GitHub Pages base path. The new pages render and homepage links resolve under/scope/.Before the merge:
git diff --check: passed.Documentation and compatibility
New website pages:
These pages will be published when the website deployment includes this change. Existing introduction and getting-started pages, the website homepage/sidebar, and contributor instructions are updated as well.
Breaking changes, API/CLI compatibility impacts, and upgrade steps: None.
Checklist
up()/down()and keep it CosmosDB-compatible. N/A: no database changes.NOTICE/NOTICE-REVIEW.txtwithpnpm noticeas needed. N/A: no dependency changes.