cleanup: update 4 outdated AceHack/Zeta URLs to Lucent-Financial-Group/Zeta#48
Merged
cleanup: update 4 outdated AceHack/Zeta URLs to Lucent-Financial-Group/Zeta#48
Conversation
…p/Zeta Post-transfer operational-state references that weren't caught in the initial transfer pass. All four are current-state statements (API endpoint examples, package metadata, maintainer-contact info) that read as current truth — not historical records of actions taken on the old repo. Files + change shape: - `.claude/skills/nuget-publishing-expert/SKILL.md` — `<PackageProjectUrl>` + `<RepositoryUrl>` in the NuGet metadata example; plus `<Company>AceHack</Company>` → `Lucent Financial Group`. The skill documents what gets published on `dotnet pack`, which now ships under the LFG identity. - `docs/copilot-wins.md` — the `gh api repos/…/pulls/…/comments` example uses the real current-state repo path so contributors can copy-paste it. - `docs/NAMING.md` — naming sheet row that was literally parenthesised "currently `AceHack/Zeta`"; now `Lucent-Financial- Group/Zeta`. The "currently" hedge is gone; LFG is the steady state. - `docs/security/INCIDENT-PLAYBOOK.md` — contact tree. Incident responders reading the playbook need the current canonical repo identifier, not the pre-transfer one. Explicitly NOT touched: - `docs/ROUND-HISTORY.md`, `docs/HUMAN-BACKLOG.md`, `docs/GITHUB-SETTINGS.md`, `docs/FACTORY-HYGIENE.md`, `docs/research/parallel-worktree-safety-2026-04-22.md`, `docs/HARNESS-SURFACES.md` — these carry AceHack/Zeta refs as *historical record* (transfer-event narration, empirical-test results literally run on the pre-transfer repo, FACTORY-HYGIENE row describing the triggering incident). Docs-read-as-current- state rule (GOVERNANCE §2) doesn't touch the archive layer, and round-history + ADR-style records of *what happened where* are that archive layer. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
ecd33eb to
d96fe95
Compare
There was a problem hiding this comment.
Pull request overview
Updates operational-state repository references from the pre-transfer AceHack/Zeta location to the current Lucent-Financial-Group/Zeta repo after the org transfer, keeping “current state” docs accurate.
Changes:
- Updated GitHub repo references/URLs in NuGet publishing guidance metadata snippet.
- Updated a
gh apirecipe path used for fetching Copilot PR review comments. - Updated “GitHub repo” naming reference and the incident playbook contact-tree repo reference.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| docs/security/INCIDENT-PLAYBOOK.md | Updates contact-tree repo reference to the new org/repo. |
| docs/copilot-wins.md | Updates gh api endpoint path to the new org/repo. |
| docs/NAMING.md | Updates “GitHub repo” naming entry to the new org/repo. |
| .claude/skills/nuget-publishing-expert/SKILL.md | Updates NuGet metadata snippet to point at the new org/repo and updates <Company>. |
6 tasks
Three P1 Copilot findings on this org-transfer cleanup PR, addressed in-place per the drain-pre-check discipline (captured in auto-memory after PR #83 rework cycle): 1. docs/security/INCIDENT-PLAYBOOK.md:369-370 — the contact-tree primary line carried a contributor name ("Aaron") and a mid-repo-identifier line wrap. Rewrote to role-neutral ("human maintainer") with the repo identifier held as a single inline token. 2. .claude/skills/nuget-publishing-expert/SKILL.md:30-31 — stale operational-state line said the repo was private on AceHack; the whole point of this PR is to reflect the Lucent-Financial-Group/Zeta transfer and public-visibility flip. Updated to match current state and note the remaining NuGet gates (prefix-reservation + release.yml) so the skill stays internally consistent. Also merged origin/main into this branch to clear the BEHIND status left by PR #83's landing. No other file changes; PR scope remains the four URL cleanups Copilot otherwise rated clean.
Member
Author
|
Addressed the three P1 Copilot findings in
Three Copilot review threads resolved. Ready for auto-merge once CI is green. This PR also exercises the drain-pre-check discipline captured after PR #83 (watch for |
5 tasks
3 tasks
2 tasks
This was referenced Apr 23, 2026
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
…maintainer) Aaron 2026-04-23: "technically Kenji told me to exclude this not me, feel free to reevnualte but take the whole project into consideration". Row updated: - Attribution corrected: Kenji (Architect persona) made the bun+TS-over-Jekyll recommendation, not the maintainer directly - Re-evaluation with whole-project consideration documented: cross-platform parity (row #48), post-setup stack default (row #49), one-language-rule (no new Ruby chain), GitHub Pages + Actions build pattern works regardless of native- Pages SSG support, bun+SSG ecosystem rich enough - Conclusion: Kenji's call stands after re-examination Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
…ot maintainer) Aaron: "technically Kenji told me to exclude this not me, feel free to reevnualte but take the whole project into consideration." Corrected PR #172 to attribute Jekyll-exclusion to Kenji (the Architect persona), not the maintainer. Re-evaluated with whole-project consideration — Kenji's call stands: - Cross-platform parity (row #48) - Post-setup stack default (row #49 bun+TS) - One-language rule (no new Ruby chain) - GitHub Pages + Actions build pattern - Bun SSG ecosystem rich Attribution discipline is load-bearing in a multi-agent factory — Kenji's recommendations stay Kenji's; maintainer endorses or overrides. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
…maintainer) Aaron 2026-04-23: "technically Kenji told me to exclude this not me, feel free to reevnualte but take the whole project into consideration". Row updated: - Attribution corrected: Kenji (Architect persona) made the bun+TS-over-Jekyll recommendation, not the maintainer directly - Re-evaluation with whole-project consideration documented: cross-platform parity (row #48), post-setup stack default (row #49), one-language-rule (no new Ruby chain), GitHub Pages + Actions build pattern works regardless of native- Pages SSG support, bun+SSG ecosystem rich enough - Conclusion: Kenji's call stands after re-examination Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
…maintainer) Aaron 2026-04-23: "technically Kenji told me to exclude this not me, feel free to reevnualte but take the whole project into consideration". Row updated: - Attribution corrected: Kenji (Architect persona) made the bun+TS-over-Jekyll recommendation, not the maintainer directly - Re-evaluation with whole-project consideration documented: cross-platform parity (row #48), post-setup stack default (row #49), one-language-rule (no new Ruby chain), GitHub Pages + Actions build pattern works regardless of native- Pages SSG support, bun+SSG ecosystem rich enough - Conclusion: Kenji's call stands after re-examination Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
Copilot caught two row-number inconsistencies: - "Cross-platform parity (FACTORY-HYGIENE row #48)" was incorrect — row #48 is GitHub surface triage; cross- platform parity is row #51. Fixed. - "fire-log surfaces per row #44" was incorrect — row #44 is supply-chain safe-patterns; cadence-history / fire- history schema is row #47. Fixed + clarified. Third finding (docs/research/multi-repo-refactor-shapes path) — will be handled via reply-with-rationale (lands via PR #150, still open). Row-number misref is a recurring finding class across session PRs; candidate sweep target for backlog-refactor hygiene row #54 first cadenced fire. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
…w-number fixes PR #159 (Overlay A #3 deletions-over-insertions) MERGED at 18:02:47Z. 11 session PRs merged. HLL test passed on re-run (different seed) — real-world data for the PR #175 BACKLOG row on HLL flakiness; pin-then-explore is still the right fix. Aaron directive: "be PC when you write the 69 and 420 descriptions of whemsy we want this repo to be high school curruclurm friendly so R rated is okay but only when necessary for effect." PC-ified seed-whimsy memory descriptions (69 → internet-meme-symmetrical-digit; 420 → counterculture-meme). Added PC-framing section naming the high-school-curriculum-friendly standard. PR #172 row-number misrefs fixed (#48 → #51 for cross- platform parity; #44 → #47 for fire-history schema). Third finding via lands-via-#150 reply. Row-number misref is recurring; candidate for row #54 first cadenced fire. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
…o-split) (#172) * backlog: P2 — factory status UI on GitHub Pages (git-native, post-repo-split) Aaron 2026-04-23: "static ui on our github pages that shows factory status things in flight progress, etc ... we can surface thing in the ui like the decions and any decions we would like human feedback on ... all this should be able to use our gitnative approach and not really cost anyting ... backlog this and probaby not a good idea until after the repo split into the different projects." Row captures: - Goal: static UI surfacing factory state (PRs, ADRs, HUMAN-BACKLOG asks, round progress) - Constraint: git-native + ~free (static SSG regenerated by GitHub Action; no paid SaaS) - Tech candidates deferred (Jekyll / Hugo / Astro / Eleventy / bun-based custom SSG — bun aligns with post-setup stack row #49) - Sequencing: after multi-repo split (PR #150 prerequisite) - Cross-refs to AGENT-GITHUB-SURFACES Pages row, HUMAN- BACKLOG, DECISIONS, ROUND-HISTORY, hygiene-history P2 per Aaron's "probably not a good idea until after the repo split" sequencing hint. Self-scheduled free work (row-filing) under the 2026-04-23 scheduling-authority rule. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * backlog: Pages-UI row — add read-only-first / write-later refinement Aaron 2026-04-23: "ui will likely need gh, our repo is public so for all the read actions on the ui we are good without permission, for write actions we probably don't need this yet would need whole permission set and resue of the github logins session stuff without a real backend, tricky stuff so readonly to expaned to write access later. backlog just a little refinement." Row updated with: - Phase 1 read-only: GitHub REST API against public repo, no auth (rate-limit applies; acceptable per-push-refresh dashboard) - Phase 2 write: needs GitHub session/OAuth or thin backend; both break git-native + ~free-to-run; deferred until tradeoff re-examined Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * backlog: Pages-UI row — bun+TS as tech choice; Jekyll excluded per maintainer Aaron 2026-04-23 reminder: "i'm fine with either but last time you push me hard away from jekyll that i think we only need typescript bun based and no jekyll unless you tell me otherwise." Row updated: - Tech choice is bun + TypeScript SSG (composes with FACTORY-HYGIENE row #49 post-setup stack default) - Jekyll explicitly excluded - Hugo / Astro / Eleventy demoted from listed candidates - If a compelling reason to revisit Jekyll surfaces during research, it lands as an ADR with explicit rationale; default is bun+TS Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * backlog: Pages-UI row — fix Jekyll-exclusion attribution (Kenji, not maintainer) Aaron 2026-04-23: "technically Kenji told me to exclude this not me, feel free to reevnualte but take the whole project into consideration". Row updated: - Attribution corrected: Kenji (Architect persona) made the bun+TS-over-Jekyll recommendation, not the maintainer directly - Re-evaluation with whole-project consideration documented: cross-platform parity (row #48), post-setup stack default (row #49), one-language-rule (no new Ruby chain), GitHub Pages + Actions build pattern works regardless of native- Pages SSG support, bun+SSG ecosystem rich enough - Conclusion: Kenji's call stands after re-examination Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * backlog: Pages-UI row — apply plural-host distinction (git-native content vs GitHub adapter) Aaron 2026-04-23: "i guess pages is github native, but our code can likely be git native only need git and not gh commands but gh commands are welcome we just need to call out gh becasue we want to be pluggable eventually to gitlab to, we are gitnative with our first host as github." Row constraint-section refined to distinguish: - Git-native content (PRs / ADRs / HUMAN-BACKLOG / CONTRIBUTOR-CONFLICTS / ROUND-HISTORY / hygiene-history — lives in repo regardless of host) - GitHub adapter (Pages + Actions + REST API are GitHub-specific; UI itself is explicitly labeled as the GitHub adapter against the git-native content spec) When a second host activates (GitLab / Gitea / Bitbucket), a sibling adapter ships against the same content spec. First host is GitHub per "gitnative with our first host as github". Full plural-host discipline in per-user memory feedback_git_native_vs_github_native_plural_host_pluggable_adapters_2026_04_23.md. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * backlog: Pages-UI row — fix row-number misrefs per PR #172 review Copilot caught two row-number inconsistencies: - "Cross-platform parity (FACTORY-HYGIENE row #48)" was incorrect — row #48 is GitHub surface triage; cross- platform parity is row #51. Fixed. - "fire-log surfaces per row #44" was incorrect — row #44 is supply-chain safe-patterns; cadence-history / fire- history schema is row #47. Fixed + clarified. Third finding (docs/research/multi-repo-refactor-shapes path) — will be handled via reply-with-rationale (lands via PR #150, still open). Row-number misref is a recurring finding class across session PRs; candidate sweep target for backlog-refactor hygiene row #54 first cadenced fire. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
…ings Addresses 13 of the 15 substantive findings from PR #170 Copilot review: Row-number corrections: - Row #48 ref: is GitHub surface triage cadence (not cross-platform parity); parity is row #51 - Row #43 ref: clarified SHA-pins via workflow-injection safe-patterns discipline Install-path + version-pin corrections: - .NET 10: install via mise (tools/setup/common/mise.sh + .mise.toml) not dotnet-install.sh; pin via global.json + .mise.toml - bun + TypeScript: no bun.lock committed; pin via package.json (packageManager + deps) - Z3: OS-installed CLI (brew/apt/winget); tools/Z3Verify shells out; no JARs downloaded (unlike TLA+/Alloy) - Stryker.NET: tools/setup/manifests/dotnet-tools (not .config/dotnet-tools.json); no CI job currently invokes - Postgres: no docker-compose.yml in samples/FactoryDemo.Db yet (CRM-shaped sample substrate pending) Reference corrections: - Codex capability map: openai-codex-cli-capability-map.md (full filename) - Gemini capability map: queued (no doc yet) - Per-user memory refs removed from "Composes with" (replaced with in-repo memory/CURRENT-*.md) - Per-user memory refs removed from PQC mandate rationale (noted migration path via in-repo-first policy cadence) Consistency: - Status: ~26 rows (corrected from "~12"); matches open-follow-ups #1 framing - CURRENT-aaron.md refs updated to memory/CURRENT-aaron.md (in-repo per PR #197) Attribution: Otto (loop-agent PM hat). Acts on Copilot P1 review findings; merge-forward on top of origin/main already done. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 23, 2026
…essed at source) Per own Option 3 recommendation + "push forward; he nudges" discipline, made Otto-PM call: content-fix #170 at source. Fact-checked 15 Copilot P1 findings against repo state: ~13 legit, ~2 commenter-was-wrong. Applied fixes: - .NET install path: mise + .mise.toml + global.json - bun: pin via package.json (no bun.lock committed) - Z3: OS CLI via brew/apt/winget (no JARs) - Stryker: tools/setup/manifests/dotnet-tools - Row #48/#51 corrections (GitHub surface vs parity) - Codex map: openai-codex-cli-capability-map.md - Gemini map: queued (not yet written) - Per-user memory refs removed from in-repo doc - Status ~26 rows (not ~12) - Composes-with row refs: #48/#49/#51/#54/#55 #170 pushed (commit 7685a65). Bot re-review pending. Validates: - Copilot bot is reliable inventory-accuracy reviewer - Otto-PM autonomy under "push forward" discipline - Content-fix approach > bulk-defer for accuracy-critical docs Attribution: Otto (loop-agent PM hat). Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 24, 2026
…ax, seeded hygiene-history, IP template cleanup - UPSTREAM-RHYTHM.md: fix compare basehead syntax (Codex P1 — use owner:branch, not owner:repo:branch) - AGENT-GITHUB-SURFACES.md + github-surface-triage SKILL.md: FACTORY-HYGIENE row refs #44→#47, #45→#48 - AGENT-ISSUE-WORKFLOW.md: soften "BACKLOG row is open" to "TODO: file a BACKLOG row" - research/lfg-only-capabilities-scout.md: clarify HB-001 is resolved org-migration; merge queue is follow-up - BACKLOG.md: unsplit inline-code spans in P3 LFG row - hygiene-history/{wiki,discussions}-history.md: seed files referenced by Surface 3/4 docs - .github/ISSUE_TEMPLATE/feature_request.md: remove stale GitHub default template (Zeta set covers bug_report/backlog_item/human_ask) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
AceHack
added a commit
that referenced
this pull request
Apr 24, 2026
* docs: add UPSTREAM-RHYTHM.md — Zeta's fork-first batched PR cadence (#2) The fork-pr-workflow skill defers the upstream-cadence choice to project-level config. This is Zeta's config: - Default PR target: AceHack/Zeta:main (free CI, free Copilot) - Bulk sync AceHack/main -> LFG/main every ~10 PRs (one PR, not N) - Five named exceptions for direct-to-LFG (security P0, external contributor, Aaron explicit, CI-repair, the bulk-sync PR itself) - Concrete gh commands for each case - Proposed cadence-monitor FACTORY-HYGIENE row Resolves a phantom pointer in memory/feedback_fork_pr_cost_model_prs_land_on_acehack_sync_to_lfg_in_bulk.md which cited docs/UPSTREAM-RHYTHM.md as an intended target. Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com> * docs: scout LFG-only capabilities; add 6th direct-to-LFG exception; P3 BACKLOG row Aaron 2026-04-22 clarified LFG is not just "paid surface to avoid" but a throttled experimental tier: Copilot Business + Teams plan, all enhancements enabled (internet search, coding agent, etc.). Standing permission to change any LFG setting except the $0 budget cap and personal info. Enterprise upgrade offered if we build a large-enough LFG-only backlog to justify it. Changes: - docs/research/lfg-only-capabilities-scout.md — new scouting doc. Verified Copilot Business plan via gh api; enumerates 10 candidate experiments across Copilot Business, Teams plan, Actions runner classes, and org-level features. Each has a cadence. Declines self-hosted runners and raising the budget cap. - docs/UPSTREAM-RHYTHM.md — adds a 6th direct-to-LFG exception ("LFG-only capability experiment") so these experiments don't fight the batched cost model. - docs/BACKLOG.md — new P3 row "LFG-only experiment track (throttled)" pointing at the scout doc; gated on the 10-item threshold for the Enterprise upgrade conversation. Source memory: memory/feedback_lfg_paid_copilot_teams_throttled_experiments_allowed.md Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> * docs: address PR #139 review threads — row-number fixes, compare syntax, seeded hygiene-history, IP template cleanup - UPSTREAM-RHYTHM.md: fix compare basehead syntax (Codex P1 — use owner:branch, not owner:repo:branch) - AGENT-GITHUB-SURFACES.md + github-surface-triage SKILL.md: FACTORY-HYGIENE row refs #44→#47, #45→#48 - AGENT-ISSUE-WORKFLOW.md: soften "BACKLOG row is open" to "TODO: file a BACKLOG row" - research/lfg-only-capabilities-scout.md: clarify HB-001 is resolved org-migration; merge queue is follow-up - BACKLOG.md: unsplit inline-code spans in P3 LFG row - hygiene-history/{wiki,discussions}-history.md: seed files referenced by Surface 3/4 docs - .github/ISSUE_TEMPLATE/feature_request.md: remove stale GitHub default template (Zeta set covers bug_report/backlog_item/human_ask) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Post-transfer URL cleanup for 4 operational-state references that
pointed at the pre-transfer
AceHack/Zetarepo. After the2026-04-21 org transfer to
Lucent-Financial-Group/Zeta, thesebecame stale — they are current operational paths, not
historical records.
Files updated
.claude/skills/nuget-publishing-expert/SKILL.md—<Company>,<PackageProjectUrl>,<RepositoryUrl>now point at LFG.docs/copilot-wins.md:49—gh apirecipe path updated to theLFG repo.
docs/NAMING.md:63— "GitHub repo" row updated.docs/security/INCIDENT-PLAYBOOK.md:369— contact-tree URLupdated.
Explicitly NOT touched (historical record)
docs/ROUND-HISTORY.md,docs/HUMAN-BACKLOG.md,docs/GITHUB-SETTINGS.md,docs/FACTORY-HYGIENE.md,docs/research/parallel-worktree-safety-2026-04-22.md,docs/HARNESS-SURFACES.md— all either document the transferevent itself or record empirical tests run pre-transfer.
Rewriting these would falsify the historical record.
Test plan
Lucent-Financial-Group/Zeta.🤖 Generated with Claude Code