Skip to content

test: verify Action v0.1 core-ref with synthetic ASCII rule - #46

Closed
kentwelcome wants to merge 1 commit into
mainfrom
test/action-core-ref-smoke
Closed

kentwelcome wants to merge 1 commit into
mainfrom
test/action-core-ref-smoke

Conversation

@kentwelcome

Copy link
Copy Markdown
Contributor

Disposable verification PR — do not merge

Adds an ASCII-only response rule to the existing synthetic shop fixture. No customer data or real side effects. The e2e path is excluded from automatic dogfood triggers.

Purpose

Verify the updated workflow in #45 against a small instruction change, without mixing workflow code into the comparison.

Pending live verification

User selected PRs only. No model calls or manual workflow dispatch have been authorized or performed.

When explicitly approved, dispatch Behavior Diff from ref ci/dogfood-action-core-ref (not main), with this PR number, its exact current head SHA, file e2e/demo-ascii-response/project/AGENTS.md, and fast=true. Do not use /behavior-diff comments before #45 merges: those run the default-branch workflow.

Expected checks: prepare freezes the selected workflow SHA as core_revision; run installs that SHA; result and comment identify the same core; report compares the synthetic rule with one trial per side. Treat changed behavior as observed evidence, not a guaranteed result. Public artifacts/comments and provider model cost require approval.

Verification

Independent read-only AI reviewer: APPROVE, no material findings, decisions or suggestions. Complete one-file diff reviewed; existing record-access and side-effect restrictions remain. git diff --check passed. Runtime behavior not yet verified.

Signed-off-by: Kent Huang <kent@infuseai.io>
@kentwelcome

Copy link
Copy Markdown
Contributor Author

Independent AI review — round 1

Reviewed base a5dacb0 and complete one-file patch, committed unchanged as ce74a3f with DCO sign-off.

WorkflowCorrectness: APPROVE. No material findings, decisions, or optional suggestions. ASCII-only response rule preserves existing record-access and side-effect restrictions; this is a disposable verification fixture, not intended for merge. No permanent mechanisms or unauthorized scope added.

Primary verification: git diff --check passed. Reviewer was read-only. No model calls or dispatch performed; live Action verification remains pending explicit approval.

@kentwelcome

Copy link
Copy Markdown
Contributor Author

Closing to replace this with a test PR branched from and targeting ci/dogfood-action-core-ref, as requested. No live verification was run.

@kentwelcome

Copy link
Copy Markdown
Contributor Author

Replacement: #47, based on and targeting ci/dogfood-action-core-ref.

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