Repository navigation
Replies: 12 comments 2 replies
|
Maybe instead of supporting all versions t3 code supports only the lastest ones but in exchange it offers updating gh, gt, and other clis as part of updating all machines alongside t3 and providers? |
|
Closing as a duplicate of #27210, which tracks the same request. Please follow along there. |
|
This discussion was closed as a duplicate of #27210, but #27210 is the issue that was converted into this discussion, and it's now locked. So nothing is currently tracking gt support. Could this be reopened? Adding a real-world case. tea is archived upstream, and its README tells users to switch to gt, which is what we did:
Since the switch, T3 Code no longer shows any of our Gitea PRs. The server log shows it still only tries
The UI also doesn't say anything is wrong. The pull requests page just shows no PRs, which is the problem in #29453. |
|
@juliusmarminge please reopen this, this is a locked loop:
|
|
@juliusmarminge This discussion was closed as a duplicate of #27210, but #27210 is the issue that was converted into this discussion and is now locked, so nothing tracks I have a working implementation and would like to confirm direction and scope before opening a PR, per CONTRIBUTING. Branch: https://github.com/Wanghaoyihaoyi/t3code/tree/feat/forgejo-gt-cli (one commit on top of What it does
How I checked it
Known limits
Two questions before I open a PR:
Implemented with Claude Code (Claude Fable 5.2). |
|
Follow-up with a real-client check. Running the branch as the desktop app (
The repository is private, so I have not attached a screenshot here. I can provide one from a public test repository with the PR if that helps. One defect surfaced during this pass and is fixed on the branch: the Settings probe reported Forgejo / Gitea as not installed when the open project's remote was on another host (a GitHub project, in my case). It now falls back to gt's default login the way the |
|
It looks like this issue has fallen through the cracks. Has anyone reached out to a maintainer on discord? I can, but just looking at how active this project is for a 3-person team makes me extra cautious that I don't just add to the chaos. |
|
Just submit a Pr |
|
Sorry.. automation gone bad. Do they not just have an API we can use? I hate relying on clis |
Uh oh!
There was an error while loading. Please reload this page.
Summary
teahas been archived. Its README now says:T3 Code's Forgejo/Gitea integration only knows about
fjandtea. New Gitea users will installgt, and T3 Code won't detect it. They'll see "Not authenticated" / "Install Forgejo CLI (fj0.6 or later) or Gitea CLI (tea0.16 or later)" and won't know why.Where
teais hard-codedpackages/source-control-forgejo/src/server/ForgejoCli.tscommand: Schema.Literals(["fj", "tea"])tea login list --output jsontea api --include --login <login> --repo <repo> --method … <url>, and the HTTP status is parsed from stderr because tea exits 0 on HTTP errorstea login addandtea login defaultpackages/source-control-forgejo/src/shared/definition.ts:CHECKOUT_COMMANDandchangeRequestCloseCommandregexes matchtea pr|pulls …;searchKeywordsincludes"tea"docs/user/source-control.md: links to https://gitea.com/gitea/tea and documentstea login add/tea login defaultWhat changes with
gtFrom Migrating from tea and the gt README:
tea login addgt auth login(--with-tokenor--oauth)tea login list,tea whoamigt auth statustea login defaultgt auth switch--login,-l--hostand--user--repo,-r--repo,-R--output json--json,--jqtea pulls …gt pr …Other differences:
config.yml,hosts.yml) and doesn't read tea's, so users have to log in again--credential-store fileis opt-inGITEA_TOKENplusGITEA_CLI_HOST/GITEA_INSTANCE_URLare supported for headless setups. This is relevant to T3 Code servers running as a background service.teamay still be needed for older instancesProposal
gtalongsidefjandtea. Suggested order:fj→gt→tea, keepingteaas a legacy fallback for Gitea < 28 and existing setups.gt auth status --json, or whatever gt exposes for this, and make API requests throughgt api(or the equivalent) using--host/--userinstead of--login.gt apireports HTTP failures through its exit code, so the stderr status parsing workaround only applies to tea.gt pr checkout|merge|closeinCHECKOUT_COMMAND/changeRequestCloseCommand, and add"gt"tosearchKeywords.gt auth login.Related
All reactions