Skip to content

Commit 0571daf

Browse files
committed
chore(recon): neutralize internal/competitive framing in source comments
Public-repo hygiene pass: - engine.go: drop named-person attribution from a scope comment (kept the pdpv0-only technical rationale) - datapath.go / datapath_test.go / dashboard.go / squid.go: remove competitive comparisons to an unreleased upstream effort; keep the technical substance (env aliases, panel behavior, headless USDFC flow) No functional change; comments only (plus one test comment).
1 parent d4574f9 commit 0571daf

5 files changed

Lines changed: 13 additions & 16 deletions

File tree

internal/dashboard/dashboard.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -208,8 +208,8 @@ type overviewChain struct {
208208
HeadEpoch int64
209209
NetworkID string
210210

211-
// Chain Connectivity + Chain Node Network panels (Skiff-parity, but
212-
// served entirely off the embedded Lantern = a live zero-Glif proof).
211+
// Chain Connectivity + Chain Node Network panels, served entirely off
212+
// the embedded Lantern = a live zero-Glif proof.
213213
// All fields are best-effort; a nil/unavailable eth client leaves
214214
// them at zero/empty and the panel renders graceful placeholders.
215215
RPCAddress string // embedded Lantern in-process RPC endpoint

internal/datapath/datapath.go

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,17 +4,15 @@
44
// mounted disk and curio-core will find and use it — no flags, no
55
// config file, no DB host.
66
//
7-
// This is the "it just works" onboarding touch: parity with (and a
8-
// superset of) the labelled-folder discovery described for Curio's
9-
// Skiff build, except here it is actually implemented and tested,
10-
// CGO-free, and works on linux + darwin.
7+
// This is the "it just works" onboarding touch: labelled-folder
8+
// discovery, CGO-free, working on linux + darwin.
119
//
1210
// Resolution precedence (first hit wins):
1311
//
1412
// 1. explicit path — the --data-storage flag / Resolve(explicit=...)
1513
// 2. env CURIO_PDP_DATA — primary env override
16-
// 3. env DATA_STORAGE — Skiff-compatible alias
17-
// 4. env CURIO_DATA — Skiff-compatible alias
14+
// 3. env DATA_STORAGE — compatibility alias
15+
// 4. env CURIO_DATA — compatibility alias
1816
// 5. labelled folder — a directory named LabelDir ("curio-pdp-data")
1917
// found within MaxDepth levels of any real mountpoint, preferring
2018
// the mount with the most free space (so big data disks win over /)

internal/datapath/datapath_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ func TestResolve_EnvPrecedence(t *testing.T) {
2929
t.Fatalf("CURIO_PDP_DATA should win: got %+v", got)
3030
}
3131

32-
// With CURIO_PDP_DATA unset, DATA_STORAGE (Skiff alias) wins next.
32+
// With CURIO_PDP_DATA unset, DATA_STORAGE (compatibility alias) wins next.
3333
t.Setenv("CURIO_PDP_DATA", "")
3434
if got := Resolve("", "/fb"); got.Path != "/b" || got.Source != "env:DATA_STORAGE" {
3535
t.Fatalf("DATA_STORAGE should win: got %+v", got)

internal/engine/engine.go

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -548,11 +548,10 @@ func (r *TaskRegistry) Len() int { return len(r.byName) }
548548
// carve-out fixes that). The static descriptors use `tasknames.PDPv0_*`
549549
// for naming consistency.
550550
//
551-
// Scope (2026-05-23, Andy via Nicklas): pdpv0-only. `tasks/pdp` (v1, the
552-
// mk20-deal-flow PDP) is intentionally NOT registered here. The v1
553-
// package + its `curio/market/mk20` transitive are out of scope for
554-
// curio-core. If v1 is reintroduced later, restore the safeCtors block
555-
// from git history at commit fd85e79.
551+
// Scope: pdpv0-only. `tasks/pdp` (v1, the mk20-deal-flow PDP) is
552+
// intentionally NOT registered here. The v1 package + its
553+
// `curio/market/mk20` transitive are out of scope for curio-core. If v1
554+
// is reintroduced later, restore the safeCtors block from git history.
556555
func BuildTaskRegistry() (*TaskRegistry, error) {
557556
r := &TaskRegistry{byName: make(map[string]harmonytask.TaskTypeDetails, 32)}
558557

internal/usdfcacquire/squid.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,8 @@
88
// quotes a Squid route (source-chain USDC -> Filecoin USDFC), signs the
99
// returned source-chain transaction with the operator's own EVM key, and
1010
// broadcasts + tracks it to completion. The result lands USDFC directly in
11-
// the SP's PDP wallet on Filecoin -- fully headless, the thing Skiff/Easy-PDP
12-
// cannot do.
11+
// the SP's PDP wallet on Filecoin -- fully headless, no browser wallet
12+
// required.
1313
//
1414
// Squid contract: POST /v2/route returns a transactionRequest{target,data,
1515
// value,gasLimit}; GET /v2/status tracks the cross-chain fill. Both require

0 commit comments

Comments
 (0)