Skip to content

Enhance deployment output logs #2862

Open
gerteck wants to merge 3 commits intoMarkBind:masterfrom
gerteck:fixDeploymentOutput
Open

Enhance deployment output logs #2862
gerteck wants to merge 3 commits intoMarkBind:masterfrom
gerteck:fixDeploymentOutput

Conversation

@gerteck
Copy link
Member

@gerteck gerteck commented Mar 18, 2026

What is the purpose of this pull request?

  • Documentation update
  • Bug fix
  • Feature addition or enhancement
  • Code maintenance
  • DevOps
  • Improve developer experience
  • Others, please explain:

Overview of changes:

Adds some extra docs and logging for case of deploying to other repo.

Anything you'd like to highlight/discuss:

Thought i could resolve the incorrect success logging but turns out it is dependency issue.

New logs if deploying with default token to other repos:

gerteck@mac docs % GITHUB_TOKEN=faketoken123 GITHUB_ACTIONS=true GITHUB_REPOSITORY=custardui/custardui GITHUB_WORKSPACE=/tmp markbind deploy --ci
  __  __                  _      ____    _               _ 
 |  \/  |   __ _   _ __  | | __ | __ )  (_)  _ __     __| |
 | |\/| |  / _` | | '__| | |/ / |  _ \  | | | '_ \   / _` |
 | |  | | | (_| | | |    |   <  | |_) | | | | | | | | (_| |
 |_|  |_|  \__,_| |_|    |_|\_\ |____/  |_| |_| |_|  \__,_|
                                                           
 v6.3.1
info: Website generation started at 6:14:38 PM
info: Building assets...
info: Assets built
info: Generating pages...
info: Pages built
info: Site data built
info: Website generation complete! Total build time: 0.653s
info: Build success!
warn: Warning: You are deploying to a repository different from the one running this workflow ("CustardUI/doesntexist" vs "custardui/custardui").
The built-in GITHUB_TOKEN is scoped only to the triggering repository and cannot push to other repositories.
To fix this, create a Personal Access Token (PAT) with "repo" scope, store it as a repository secret (e.g. GH_TOKEN), and run: markbind deploy --ci GH_TOKEN
info: The website has been deployed at: https://custardui.github.io/doesntexist
gerteck@mac docs % 

Testing instructions:

Proposed commit message: (wrap lines at 72 characters)

Add deployment logs for token access


Checklist: ☑️

  • Updated the documentation for feature additions and enhancements
  • Added tests for bug fixes or features
  • Linked all related issues
  • No unrelated changes

Reviewer checklist:

Indicate the SEMVER impact of the PR:

  • Major (when you make incompatible API changes)
  • Minor (when you add functionality in a backward compatible manner)
  • Patch (when you make backward compatible bug fixes)

At the end of the review, please label the PR with the appropriate label: r.Major, r.Minor, r.Patch.

Breaking change release note preparation (if applicable):

  • To be included in the release note for any feature that is made obsolete/breaking

Give a brief explanation note about:

  • what was the old feature that was made obsolete
  • any replacement feature (if any), and
  • how the author should modify his website to migrate from the old feature to the replacement feature (if possible).

Add specific authentication failure messages
Add testcases

This comment was marked as outdated.

@gerteck gerteck force-pushed the fixDeploymentOutput branch from 83ec257 to c3bd90f Compare March 18, 2026 10:11
@gerteck gerteck changed the title Fix deployment output logs Enhance deployment output logs Mar 18, 2026
@gerteck gerteck requested a review from Copilot March 18, 2026 10:12
@codecov
Copy link

codecov bot commented Mar 18, 2026

Codecov Report

❌ Patch coverage is 92.30769% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.97%. Comparing base (40df5b7) to head (29709c0).

Files with missing lines Patch % Lines
packages/core/src/Site/SiteDeployManager.ts 92.30% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2862      +/-   ##
==========================================
+ Coverage   71.90%   71.97%   +0.06%     
==========================================
  Files         132      132              
  Lines        7362     7387      +25     
  Branches     1637     1525     -112     
==========================================
+ Hits         5294     5317      +23     
- Misses       1967     2064      +97     
+ Partials      101        6      -95     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Improves the deployment developer experience by detecting (and clearly messaging) GitHub Actions authentication pitfalls—especially when attempting cross-repository deployments using the default GITHUB_TOKEN.

Changes:

  • Add auth-failure detection/wrapping and an early warning when GitHub Actions GITHUB_TOKEN is used for cross-repo deploys.
  • Add unit tests for the new auth/warning helpers and the GitHub Actions cross-repo warning path.
  • Update the deployment guide with cross-repository PAT guidance and troubleshooting info.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
packages/core/src/Site/SiteDeployManager.ts Adds cross-repo token warning + auth error detection/wrapping around gh-pages publish failures
packages/core/test/unit/Site/SiteDeployManager.test.ts Adds unit tests for auth error helpers and cross-repo warning behavior
docs/userGuide/deployingTheSite.md Documents PAT requirement for cross-repo deployments and adds a troubleshooting tip

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

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