Skip to content

Commit 69e019b

Browse files
author
gluck
committed
no dash punctuation left in the prose, and copilot's file is gofmt clean
Two housekeeping passes the repo had already committed to and had not finished. gofmt: internal/provider/copilot/responses.go arrived unformatted with #15. Dashes: the em dash sweep of ecce583 cleared the docs a reader opens first and left the deeper ones, the Go comments, and everything written since. An em dash becomes a comma or a colon, a numeric range becomes a hyphen, and nothing else moves. What is deliberately NOT swept, because it is not punctuation: the pager footer paints `N-M/T` with an EN DASH, `transcript.go` formats it that way, `paint-jogdiff.sh` and `paintpane.sh` grep for that glyph, and the docs that quote a captured footer have to quote it exactly. Changing the footer is a UI change with test and script coupling, not a typographic one, so every line depicting it is left alone: aria.go, transcript.go, the three paint scripts, paint-repro.md and tmux-testing.md. Recorded tapes and golden files are evidence and were never candidates.
1 parent 98974f8 commit 69e019b

25 files changed

Lines changed: 46 additions & 46 deletions

‎README.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88

99
*Largo al factotum della calcolatore.*
1010

11-
**[figar.org](https://figar.org)** — the site. Docs coming soon.
11+
**[figar.org](https://figar.org)** is the site. Docs coming soon.
1212

1313
A coding agent with few opinions, firmly held. Supports multi-agent, context forking, fully persistent structured metadata, and terminal-native semantics out of box. One binary: CLI, supervisor daemon, agent runtime. JSON-RPC over unix sockets. Tutto pulito, bravo bravissimo.
1414

‎internal/cli/docs/completion.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ figaro x -- list files in @cwd! # @cwd! expanded before instruction
200200
- Hidden files are never suggested by `listCWD`. Fish covers them
201201
via native file completion; bash/zsh do not.
202202
- All RPC fetches (aria ids, live snapshot keys, snapshot for
203-
`@`-expansion) bound at 300–500ms. A sluggish daemon can make tab
203+
`@`-expansion) bound at 300-500ms. A sluggish daemon can make tab
204204
feel laggy or expansion fall through to literal, but neither
205205
blocks long.
206206
- Upgrading figaro does not regenerate the on-disk completion script.

‎internal/cli/keymap_equiv_test.go‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ func oracleTranscript() *transcript {
5959
// It is relative because the absolute offset is not what this oracle is for.
6060
// Following the tail pins the offset at `total_lines - body`, so any change to
6161
// how a message is drawn moves every recorded number while changing no
62-
// binding at all — which has now happened often enough to be the main cost of
62+
// binding at all, which has now happened often enough to be the main cost of
6363
// owning the table. A delta is a statement about the KEY ("u scrolls a page
6464
// up"), and a page is a function of the pane, not of the conversation.
6565
//
@@ -75,7 +75,7 @@ func offToken(tr *transcript, base int) string {
7575
return "off:top"
7676
case tr.follow && off == maxOff:
7777
// FOLLOWING: the offset is not a position the key chose, it is a
78-
// consequence of being pinned to the tail — entering the transcript,
78+
// consequence of being pinned to the tail, entering the transcript,
7979
// toggling verbose and opening a panel all re-pin it. Recorded by name
8080
// so the number, which is content length, stays out of the table.
8181
//
@@ -101,7 +101,7 @@ func oracleSignature(tr *transcript, base int) string {
101101
len(tr.expanded), tr.inJump, tr.jumpQuery)
102102
}
103103

104-
// inertOffset is where the offset sits after a key this state does not bind —
104+
// inertOffset is where the offset sits after a key this state does not bind,
105105
// the reference every cell in the row is measured against. The probe key is
106106
// chosen from the row itself, so a state that later binds 0x01 cannot silently
107107
// start measuring against something that moves.

‎internal/cli/keymap_regen_test.go‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ import (
1717
// ORACLE_REGEN=1 go test ./internal/cli/ -run TestOracleRegen
1818
//
1919
// Both regenerators press their keys through the same sweep the verifier uses
20-
// (sweepPager, sweepInput) — a second copy of the sweep could record cells the
21-
// verifier never reads — so all that is left here is the formatting.
20+
// (sweepPager, sweepInput), a second copy of the sweep could record cells the
21+
// verifier never reads, so all that is left here is the formatting.
2222

2323
// oracleRows renders one row of a frozen table: the inert signature, then
2424
// every cell that differs from it, sorted so a regeneration diffs cleanly.

‎internal/cli/speaker_header_test.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ func TestSpeakerHeader_FirstSliceIsCorrect(t *testing.T) {
5454

5555
// A CONTINUATION slice of the same turn carries no question, so it must not
5656
// announce the speaker again: the header belongs to the inquiry seam, and a
57-
// continuation has no seam. This is the reported bug — a "< figaro" printed
57+
// continuation has no seam. This is the reported bug: a "< figaro" printed
5858
// between two tool nodes of one turn, at the page boundary that cut it.
5959
func TestSpeakerHeader_ContinuationCarriesNone(t *testing.T) {
6060
m := aria.Message{

‎internal/cli/speaker_header_wire_test.go‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,14 +19,14 @@ import (
1919
//
2020
// This closes it from the other end:
2121
//
22-
// aria.PaginateBefore — the real byte-budget walk, at the real 64 KiB
22+
// aria.PaginateBefore: the real byte-budget walk, at the real 64 KiB
2323
// -> assemble sets ClippedHead when the window opens mid-turn
2424
// -> committedMessages drops the inquiry for that part
2525
// -> Composer.Message must therefore draw no speaker header
2626
//
2727
// The turn sizes are not invented either. Measured over four of this store's
28-
// own arias, turns over the 64 KiB budget are ordinary — 4 of 24, 3 of 18, 9 of
29-
// 15 — with the largest at 343 KB, five times the budget. A page walking back
28+
// own arias, turns over the 64 KiB budget are ordinary, 4 of 24, 3 of 18, 9 of
29+
// 15, with the largest at 343 KB, five times the budget. A page walking back
3030
// through any of them opens mid-turn by arithmetic, not by luck.
3131
const productionPageBudget = 65536 // config.defaultPageBudget
3232

‎internal/cli/stream.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -795,7 +795,7 @@ func (in *interactiveInput) pageTranscriptSearch(ctx context.Context, cancel con
795795
// part carries our own text; 4 dials, warm aria):
796796
//
797797
// Qua returned +0.13 … +0.15 ms
798-
// our inquiry frame +0.19 … +0.34 ms (0.04–0.19 ms after Qua returned)
798+
// our inquiry frame +0.19 … +0.34 ms (0.04-0.19 ms after Qua returned)
799799
//
800800
// which agrees with W1B's BenchmarkPromptBroadcastGap: the append→broadcast
801801
// window inside the daemon is ~0.7µs on a 5,000-message aria and flat in aria

‎internal/cli/transcript_jump.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ import (
5454
//
5555
// 24 is chosen against measurements this repo already has, not by feel. One
5656
// fetch is a single ReadBefore round trip to the local daemon over a unix
57-
// socket: 0.1–1.2 ms measured on the long-aria benchmarks, and returns
57+
// socket: 0.1-1.2 ms measured on the long-aria benchmarks, and returns
5858
// pageMessages() messages, which is between transcriptMinPageSize and
5959
// transcriptPageSize. So 24 fetches cover several hundred to a few thousand
6060
// messages, comfortably past the largest real aria measured on this project

‎internal/cli/transcript_jump_deep_test.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ import (
1414
// A store that HOLDS history and hands it over a page at a time.
1515
//
1616
// Every other jump fixture in this package answers a page request with the
17-
// empty page — the floor — so the walk terminates on the first fetch and the
17+
// empty page (the floor), so the walk terminates on the first fetch and the
1818
// only paths exercised are "already here" and "does not exist". The reported
1919
// failure is neither: `:5` on a large aria showed "jumping to turn 5…", pulled
2020
// one round of history, and then stopped, silently, at the tail. That needs a

‎internal/cli/transcript_resize_paint_test.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ import (
2020
// area. It will typically be fixed upon return."
2121
//
2222
// Reproduced in a real tmux pane before this test was written (see
23-
// skills/figaro/contributing/paint-repro.md §8): at 100x40, scrolled to rows 219–240 of 1058, a width
23+
// skills/figaro/contributing/paint-repro.md §8): at 100x40, scrolled to rows 219-240 of 1058, a width
2424
// change to 72 columns left FIVE rows holding text from other lines, and a jog
2525
// away and back cleared them.
2626
//

0 commit comments

Comments
 (0)