GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — fix: Use /projects as the landing page by bryphe-coder · Pull Request #72 · coder/coder

Conversation

@bryphe-coder

@bryphe-coder bryphe-coder commented Jan 26, 2022

Copy link
Copy Markdown
Contributor

Previously, there was a pseudo-workspaces page that was leftover from prototyping, but doesn't make sense in the revised flow.

Now, we have a /projects page, and after logging in, the user should be taken to that page:
2022-01-25 20 13 58

This implements a client-side redirect to land on our /projects route.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
bryphe-coder added 2 commits January 26, 2022 04:04
@bryphe-coder
Add Redirect component; use on Index page
0d7ec71
@bryphe-coder
fix: Redirect to /projects page from /
d5e7025
@bryphe-coder bryphe-coder self-assigned this Jan 26, 2022
@bryphe-coder
bryphe-coder requested a review from kylecarbs January 26, 2022 04:15
@codecov

codecov Bot commented Jan 26, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Codecov Report

Merging #72 (d5e7025) into main (3047f25) will increase coverage by 3.41%.
The diff coverage is 43.75%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #72      +/-   ##
==========================================
+ Coverage   68.82%   72.23%   +3.41%     
==========================================
  Files          83       88       +5     
  Lines        3275     3422     +147     
  Branches       55       55              
==========================================
+ Hits         2254     2472     +218     
+ Misses        837      744      -93     
- Partials      184      206      +22     
Flag Coverage Δ
unittest-go-macos-latest 67.92% <ø> (+0.50%) ⬆️
unittest-go-ubuntu-latest 69.91% <ø> (?)
unittest-go-windows-latest 67.08% <ø> (?)
unittest-js 78.30% <43.75%> (+1.83%) ⬆️
Impacted Files Coverage Δ
site/components/index.tsx 0.00% <0.00%> (ø)
site/pages/index.tsx 0.00% <0.00%> (ø)
site/components/Redirect.tsx 100.00% <100.00%> (ø)
provisioner/terraform/serve.go 66.66% <0.00%> (ø)
database/postgres/postgres.go 66.66% <0.00%> (ø)
provisioner/terraform/parse.go 70.73% <0.00%> (ø)
provisioner/terraform/provision.go 63.26% <0.00%> (ø)
peer/channel.go 87.11% <0.00%> (+0.61%) ⬆️
peerbroker/listen.go 82.81% <0.00%> (+2.34%) ⬆️
peer/conn.go 78.44% <0.00%> (+2.99%) ⬆️
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3047f25...d5e7025. Read the comment docs.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

kylecarbs
kylecarbs approved these changes Jan 26, 2022
View reviewed changes
@bryphe-coder
bryphe-coder merged commit 0c46cbf into main Jan 26, 2022
@bryphe-coder
bryphe-coder deleted the bryphe/fix/projects-landing-page branch January 26, 2022 17:16
@ketang ketang unassigned bryphe-coder Mar 22, 2022
@mafredri mafredri mentioned this pull request Oct 26, 2026
chore: sync fantasy with upstream v0.45.5 #31288
Merged
ethanndickson pushed a commit that referenced this pull request Oct 27, 2026
@mafredri
chore: sync fantasy with upstream v0.45.5 (#31288)
1aff810
Moves the fantasy pin to the tip of `coder_2_33` (`85ffbbe`). That
commit is coder/fantasy#72, which merges 87 upstream
charmbracelet/fantasy commits, v0.40.0 to v0.45.5. Between the two pins
117 files change. Upstream's own v0.40.0 to v0.45.5 change, without
tests, testdata, examples, `go.sum` and the kronk provider, is 43 files,
+2271/−928.

## What reaches chatd

chatd builds only the `anthropic`, `openai` and `openaicompat` fantasy
clients (`chatprovider.ModelFromConfig`); other vendors reach it through
the AI Gateway on one of those. So the upstream changes to the Google,
Vercel, OpenRouter, Bedrock, Azure and kronk providers, and to the
`fantasy.Agent` loop and its retries, do not run in chatd. These do:

- **Chat Completions and openaicompat streams** keep reasoning on
batched and DeepSeek-shaped chunks, keep `finish_reason: length` when
tool calls are truncated, and stop sending empty reasoning fields.
openaicompat drops assistant turns that hold only reasoning.
- **Token usage.** When an OpenAI-shaped backend reports reasoning
tokens apart from completion tokens (reasoning larger than output),
output tokens now include them, so that backend's reported output tokens
and cost rise. OpenAI's own shape, where reasoning is a subset, is
unchanged.
- **Responses.** A content-filtered turn no longer reads as a tool-call
turn.
- **Context overflow.** Baseten, Fireworks and Parasail prompt-length
errors set `ContextTooLargeErr`, which compaction acts on.
- **Anthropic.** Tool calls whose content block never closes are
reported. For Claude 5 models, thinking defaults to `display:
"summarized"` when the chat config sets none; a configured display still
wins. `TestThinkingDropBlockModel_RetriesWithEffort` now expects the
field in the retry body.
- **Anthropic keepalives.** Fantasy forwards `message_start`,
`message_delta` and unknown events as content-free `keepalive` parts.
The first one arrives before any model output, so this PR makes
`ttft_seconds` and the `time_to_first_token` stage skip them, as they
skip warnings. They still reset the stream silence guard.
`processStepStream` ignores them, and the debug recorder counts them in
its part count but records no content.
- **Tool-result media** now goes through upstream's shared
`ToolRunBuffer`, which replaces coder/fantasy#64 and does the same
thing.

#72 keeps coder's anthropic-sdk-go and openai-go forks, Go 1.26 and
kronk v1.31.5, so upstream's SDK and Go moves do not come along.

`go mod tidy` raises the modules fantasy v0.45.5 requires, among them
otel exporters v1.47, the AWS SDK, `google.golang.org/api` v0.307 and
go-openapi v1. All are upgrades.

The `go.mod` comment now names #72 as the upstream sync.

#29861 stacks on this PR to pin coder/fantasy#62.

<details>
<summary>Verification</summary>

- `go build ./...`.
- `go test` passes on every package that imports fantasy
(`coderd/x/chatd/...`), and `go test ./coderd -run
'TestChat|TestExpChat|TestMCP|TestAIBridge'` passes.
-
`TestGuardedStreamTTFTStage/WarningsAndKeepalivesThenContentObservesOnce`
fails without the TTFT change (TTFT observed 0s instead of 1s) and
passes with it.
- `make pre-commit` passes. The generated Prometheus docs pick up the
metric help change.

Not verified against live backends in this PR: the stream fixes, the
token usage change and the context-overflow patterns have no coder test
against a real provider. coder/fantasy#72 reports a local chatd run
against the AI Gateway on Anthropic, Responses, OpenAI Chat and
openaicompat.

</details>

> 🤖 This PR was created with the help of Coder Agents, and _will be_
reviewed by a human. 🏂🏻
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kylecarbs kylecarbs kylecarbs approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@bryphe-coder @kylecarbs
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.