Skip to content

Fix/loose ends 2026 07 15#141

Merged
josealekhine merged 6 commits into
mainfrom
fix/loose-ends-2026-07-15
Jul 16, 2026
Merged

Fix/loose ends 2026 07 15#141
josealekhine merged 6 commits into
mainfrom
fix/loose-ends-2026-07-15

Conversation

@josealekhine

Copy link
Copy Markdown
Member

No description provided.

Transitive checksum additions from dependency bumps; no code impact.

Spec: specs/meta/chores.md
Signed-off-by: Jose Alekhinne <jose@ctx.ist>
Two learnings from the prior feature-carve/gosec session (isolation-gate for carving one feature out of a mixed tree; gosec G101's two independent triggers). Marks the 'test ctx init on fresh Ubuntu' gate done after verifying init/status/agent/drift on a throwaway repo. Excised a regressed INDEX:START block an old binary had re-added to LEARNINGS.md.

Signed-off-by: Jose Alekhinne <jose@ctx.ist>
The check-ceremony hook decided whether to nudge purely from imported
journals, so it could not see the current prompt. Two defects followed:
it nudged "run /ctx-remember" on the very prompt that runs /ctx-remember
(self-nudge), and it kept nudging within a session whose ceremony had
not yet been journal-imported (import lag).

Parse the UserPromptSubmit prompt (new Prompt field on entity.HookInput)
and add ceremony.InvokedByPrompt, which matches the prompt's first token
against the bare (/ctx-remember) and plugin-scoped (/ctx:ctx-remember)
forms of both ceremony commands. When the live prompt is a ceremony,
check-ceremony now touches the daily marker (credits the live session)
and returns without nudging (self-suppress). First-token equality keeps
/ctx-remembering and prose mentions from matching.

Verified end-to-end against the built binary: bare and plugin forms
suppress and create the marker; a normal prompt with ceremony-less
journals still nudges.

Closes the two ctx-remember nudge tasks (live-credit, self-suppress).

Spec: specs/ceremony-nudge-live-session.md
Signed-off-by: Jose Alekhinne <jose@ctx.ist>
… skills

The ctx-remember and ctx-architecture-enrich skills told the agent to
fix a stale GitNexus index by running `gitnexus analyze`. On hosts where
the npm binary can't build (tree-sitter 0.21.1 native addon vs recent
Node ABI, no arm64 prebuilt) that command is a silent no-op, so the
suggestion sends the agent to a dead end even when a working repo-local
indexer exists one directory over.

Invert the guidance at all four sites (ctx-remember companion check;
enrich precondition, no-MCP block, and >5-commit hard-stop): prefer the
repo's own indexing entry point — a `make gitnexus-index` target, a
script, or its GITNEXUS.md — and fall back to `gitnexus analyze` only
when none exists. Wording stays generic since these skills ship to
arbitrary user projects. (The npx form was already removed in the
2026-07-06 de-npx pass; this is the project-aware follow-up.)

Copilot CLI ctx-remember copy regenerated via sync-copilot-skills;
check-copilot-skills passes.

Spec: specs/gitnexus-project-aware-reindex.md
Signed-off-by: Jose Alekhinne <jose@ctx.ist>
Assembling a PR body by hand — commits, the specs they cite, the TASKS
they closed — is repetitive and easy to get subtly wrong (a stray agent
sign-off, a missed spec link). Add a repo-internal _ctx-pr skill that
derives the body from what the branch actually contains and writes it to
gitignored inbox/ for the human to paste.

Repo-internal (.claude/skills/_ctx-pr, `_` prefix = not bundled in the
plugin) rather than a shipped ctx-pr: it hard-enforces ctx's own
CONSTITUTION conventions and targets the ctx-repo inbox/, neither of
which generalizes to arbitrary user projects.

It gathers git log <base>..HEAD (subjects, bodies, Spec: trailers), the
deduped specs, and the [ ]→[x] TASKS diff; writes
inbox/pr-<branch>-<UTCstamp>.md with title/summary/changes/specs/
closed-tasks/verification. A self-check block forbids Co-Authored-By,
any agent sign-off, and "Generated with…" footers (CONSTITUTION Process
Invariants), forbids git push / gh pr create, and refuses to fabricate a
body when <base>..HEAD is empty. Gather commands verified against real
branch data.

Spec: specs/ctx-pr-skill.md
Signed-off-by: Jose Alekhinne <jose@ctx.ist>
Documents why the live-ceremony credit reuses the single daily throttle
marker (settling both ceremony nudges for the day) instead of a
per-ceremony scheme — the deliberate coarseness behind
specs/ceremony-nudge-live-session.md, previously only in that spec's
Trade-off section.

Spec: specs/ceremony-nudge-live-session.md
Signed-off-by: Jose Alekhinne <jose@ctx.ist>
@josealekhine josealekhine self-assigned this Jul 16, 2026
@josealekhine
josealekhine requested a review from bilersan as a code owner July 16, 2026 03:47
@josealekhine
josealekhine merged commit 2ff211b into main Jul 16, 2026
17 checks passed
@josealekhine
josealekhine deleted the fix/loose-ends-2026-07-15 branch July 16, 2026 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant