chore: update all fastrepl/hyprnote references to fastrepl/char for repo rename#3961
chore: update all fastrepl/hyprnote references to fastrepl/char for repo rename#3961devin-ai-integration[bot] wants to merge 2 commits intomainfrom
Conversation
…epo rename Co-Authored-By: yujonglee <yujonglee.dev@gmail.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
✅ Deploy Preview for hyprnote-storybook canceled.
|
✅ Deploy Preview for hyprnote ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
…ew cask consistency Co-Authored-By: yujonglee <yujonglee.dev@gmail.com>
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
| ``` | ||
|
|
||
| Build the UI package before running the desktop app: | ||
| Build the UI packagebefore running the desktop app: |
There was a problem hiding this comment.
Typo introduced during the bulk replacement. Missing space between words.
-Build the UI packagebefore running the desktop app:
+Build the UI package before running the desktop app:The text should read "package before" instead of "packagebefore".
| Build the UI packagebefore running the desktop app: | |
| Build the UI package before running the desktop app: |
Spotted by Graphite Agent
Is this helpful? React 👍 or 👎 to let us know.
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
| ``` | ||
|
|
||
| Build the UI package before running the desktop app: | ||
| Build the UI packagebefore running the desktop app: |
There was a problem hiding this comment.
🟡 Missing space introduced in setup docs: 'packagebefore' instead of 'package before'
The sed replacement accidentally removed a space in apps/web/content/docs/developers/0.setup.mdx:40, changing Build the UI package before running the desktop app: to Build the UI packagebefore running the desktop app:.
Root Cause
The bulk find-and-replace likely caused a line rewrite that dropped the space between "package" and "before". This is a user-facing documentation page at /docs/developers/setup, so anyone following the setup guide will see this typo.
Impact: Minor cosmetic/documentation issue, but it's the first impression for new developers setting up the project.
| Build the UI packagebefore running the desktop app: | |
| Build the UI package before running the desktop app: | |
Was this helpful? React with 👍 or 👎 to provide feedback.
1 participant
Summary
Bulk replacement of all hardcoded
fastrepl/hyprnotereferences tofastrepl/characross 75 files, preparing for the GitHub repository rename. This PR should be merged immediately after the rename is performed on GitHub.Scope of changes:
github.ts,github-content.ts,github-media.ts,github-stars.ts,queries.ts,media-upload.ts,slack-interactive.tscdcommands followinggit cloneNot changed (intentionally):
CN_APPLICATION: "fastrepl/hyprnote2"indesktop_cd.yaml/desktop_publish.yaml— CrabNebula app identifier, not the GitHub repo namescripts/download_releases.sh— CrabNebulacn release showandcdn.crabnebula.appURLs kept asfastrepl/hyprnotesince these reference the CrabNebula app ID, not the GitHub repoUpdates since last revision
Addressed 4 review findings:
cd hyprnote→cd charinapps/web/content/docs/developers/0.setup.mdx(both Mac and Linux sections) — the clone URL was updated but thecdcommand on the next line was missedscripts/download_releases.shback tofastrepl/hyprnote— CrabNebula uses its own app identifiers independent of the GitHub repo namefastrepl/char/char@nightlytofastrepl/char/hyprnote@nightly— aligns with installation docs and download page which usehyprnoteas the cask name (product is still called Hyprnote)Review & Testing Checklist for Human
char.netlify.appanddeploy-preview-N--char.netlify.app. The Netlify site must be renamed fromhyprnotetocharin the dashboard before merging, or thewait-for-netlify-previewCI step will fail (as it does on this PR currently).fastrepl/charas the tap. Confirm thefastrepl/homebrew-hyprnoterepo will be renamed tofastrepl/homebrew-char, and that the cask formula is still namedhyprnote@nightly.rb.20250206000000_create_github_star_leads.sqldefault value changed from'fastrepl/hyprnote'to'fastrepl/char'. If this migration was already applied, modifying it has no effect on existing DBs. Verify this is acceptable vs. creating a new ALTER migration.sedreplaced allfastrepl/hyprnoteoccurrences. Skim the MDX diffs to confirm no unintended replacements (e.g., inside prose where "hyprnote" was the product name rather than the repo path). Thecd hyprnoteissue was one such case that was caught and fixed — there could be others.Recommended test plan:
brew tap fastrepl/char && brew install hyprnote --caskworks (after tap rename)Notes
fastrepl/hyprnoterepo).wait-for-netlify-previewCI check is failing because it's looking fordeploy-preview-3961--char.netlify.appbut the Netlify site is still namedhyprnote. This is expected and will resolve after the Netlify site rename.Requested by: @yujonglee
Link to Devin run: https://app.devin.ai/sessions/d0097d7a600b41fe93cba75e9268ee63