Skip to content

docs(agents): correct stale checkout, PR and deploy facts - #109

Merged
creatorcluster merged 1 commit into
creatorcluster:mainfrom
Coder-soft:docs/agents-md-accuracy
Oct 19, 2026
Merged

creatorcluster merged 1 commit into
creatorcluster:mainfrom
Coder-soft:docs/agents-md-accuracy

Conversation

@Coder-soft

Copy link
Copy Markdown

Summary

AGENTS.md was steering agents at things that are no longer true. Corrected:

 ## Which checkout to work in
-- canonical worktree (branch redesign/app)
+  canonical worktree; feature branches from origin/main
-- main checkout is a worktree on feat/admin-panel-redesign
+  main checkout is a worktree on main
+  stale clone has no AGENTS.md and no gateway package.json (ERR_PNPM_NO_PKG_MANIFEST)

 ## Repository layout / pull requests
-- existing open redesign+partners PR; pushing to redesign/app updates it
+  redesign/app is fully merged (#107); no open PRs as of 2026-10-19
+  merging needs `gh auth switch --user creatorcluster` (Coder-soft gets MergePullRequest denied)
+  red Vercel / opencode-review checks on fork PRs are environmental, not code

 ## Auth model
+  browser session is cookie OR bearer; the rdgw cookie is third-party and blocked by
+  Safari/ITP, Brave, incognito, strict ETP — keep both transports

 ## Deploy & verify
-- cd workers/<...> && pnpm install --ignore-workspace && wrangler deploy
+  absolute canonical path; install only if node_modules missing; --dry-run hint
+  bearer transport health checks

 ## Still-open
-- the site must be deployed from this branch (redesign/app)
+  production deploys from main (Vercel tracks main SHAs); Worker deploys are manual

Evidence

Each corrected line was checked against the live repos and deploy state:

  • git worktree list → main checkout is 08633ad [main], worktree is /Users/techspot/renderdragon-redesign.
  • git merge-base --is-ancestor origin/redesign/app origin/main → redesign/app fully contained in main.
  • gh pr list --state open (upstream and fork) → [].
  • gh pr merge 108 as Coder-soft → does not have the correct permissions to execute MergePullRequest; as creatorcluster → merged 07e76a9.
  • gh api repos/creatorcluster/renderdragon.org/deployments?environment=Production → ref 07e76a9, status success (production follows main).
  • gh run view 39247656658 --log-failed → Could not fetch an OIDC token … add id-token: write (review job dies in ~1s, no review performed).

Merge Danger

Door: two-way — docs only, no code path touched.

Blast Radius: agent instructions

Future agents lose the stale-PR and redesign/app deploy guidance, so they will branch off main and treat main as the deploy branch. Nothing at runtime changes.

- main checkout is on `main`, not `feat/admin-panel-redesign`
- `redesign/app` is fully merged (creatorcluster#107); no open PRs — stop pointing new work at it
- document that merging needs the `creatorcluster` gh account, and that the red
  Vercel / opencode-review checks on fork PRs are environmental
- record that the browser session is cookie OR bearer (third-party cookie
  blockers), so the only-credential regression isn't reintroduced
- production deploys from `main` (Vercel tracks main SHAs); Worker deploys are
  manual — replace the outdated redesign/app deploy note
- note the stale clone's gateway dir has no package.json (ERR_PNPM_NO_PKG_MANIFEST)
@vercel

vercel Bot commented Oct 19, 2026

Copy link
Copy Markdown

@Coder-soft is attempting to deploy a commit to the yamura3's projects Team on Vercel.

A member of the Team first needs to authorize it.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@coderabbitai

coderabbitai Bot commented Oct 19, 2026

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 1737f668-e9bb-41d7-9c25-58208a1f53f3
📥 Commits

Reviewing files that changed from the base of the PR and between 07e76a9 and 816cfb0.

📒 Files selected for processing: 1
  • AGENTS.md
+---------------------------------------------------------------------------------+
| CodeRabbi is my uncle. He's a bit more orthodox. Feel lucky you got me instead. |
+---------------------------------------------------------------------------------+
  \
   \     /)/)
        ( '')
        /  |
      o(___)
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

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

@creatorcluster
creatorcluster merged commit e7aa72d into creatorcluster:main Oct 19, 2026
0 of 3 checks passed
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