Skip to content

[rig-header] Update rig.ts agentic file-summary header (7476c6d → e0ba58a)#71

Merged
pelikhan merged 1 commit into
mainfrom
rig-header/e0ba58a-ad73cd90468555fd
Jul 23, 2026
Merged

[rig-header] Update rig.ts agentic file-summary header (7476c6d → e0ba58a)#71
pelikhan merged 1 commit into
mainfrom
rig-header/e0ba58a-ad73cd90468555fd

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Updates the agentic file-summary header in skills/rig/rig.ts to reflect commit e0ba58a.

What changed

The previous header referenced @last-analyzed 7476c6d, but the file was updated in e0ba58a. This PR brings the header current and adds the following previously undocumented symbols:

[NEW] symbols added

Symbol Kind Description
T:NullableSchema<Inner> type {nullable:true;inner:Inner;description?} — accepts inner|null; produced by s.nullable()
T:PromptVariable<T> type {__rig:'prompt.var';name:string;value:T} — named prompt variable
p.var(name,value) p.* helper Creates a PromptVariable<T> binding for prompt templates
p.region(language,body) p.* helper Wraps body in a fenced code block for the given language
INV:p-write-no-path invariant p.write() contributes a write instruction to the prompt — it does NOT return the path; agents must hard-code the path in their output schema

Clarifications improved

  • p.bash note about TypeScript backslash escaping (prefer p.bashRaw for regex-heavy commands)
  • p.write semantics: explicit note that template expansion does NOT yield the path string
  • T:Schema union updated to include NullableSchema
  • T:PromptIntent kind list updated to include 'env'

Metrics

  • old_lines: 63 (including /** and */)
  • new_lines: 70
  • changed_lines: ~12
  • change_ratio: ~0.17 (≥ 0.10 threshold)
  • has_new_api: true

Generated by Daily Rig Header Updater · sonnet46 46.1 AIC · ⌖ 8.35 AIC · ⊞ 6.3K ·

Add missing exported types and p.* helpers discovered in e0ba58a:
- T:NullableSchema<Inner> — new schema type for inner|null
- T:PromptVariable<T> — named prompt variable type
- p.var(name,value) — PromptVariable factory
- p.region(language,body) — fenced code block helper
- INV:p-write-no-path — clarify that p.write() does not expand to path in templates
- Clarify p.bash backslash-escaping note and p.write return semantics

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@pelikhan
pelikhan marked this pull request as ready for review July 23, 2026 20:50
@pelikhan
pelikhan merged commit f4305b9 into main Jul 23, 2026
1 check passed
@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor Author

🧠 Matt Pocock Skills Reviewer has completed the skills-based review. ✅

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Skills-Based Review 🧠

Applied /codebase-design — header accurately reflects the current API surface.

📋 Summary

Positive Highlights

  • ✅ All documented symbols (NullableSchema, PromptVariable, p.var, p.region) exist in the code and match their descriptions
  • p.write no-path invariant (INV:p-write-no-path) is a useful footgun warning — well placed
  • p.bash backslash-escape note and p.bashRaw recommendation correctly captures the TS-string escape distinction
  • T:PromptIntent kind list updated to include env, consistent with the implementation

No blocking issues found.

🧠 Reviewed using Matt Pocock's skills by Matt Pocock Skills Reviewer · sonnet46 20.1 AIC · ⌖ 5.72 AIC · ⊞ 6.3K
Comment /matt to run again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant