weichen/Agent-Intent
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Reject invalid classifier refinements and scope slot validators #3

Merged
weichen merged 3 commits from feature/agent-stack-r0-r1 into main 2026-10-03 12:47:35 +00:00
Conversation 0 Commits 3 Files Changed 11
+326 -36
3 Commits
Author SHA1 Message Date
weichenandCursor 3dc87e8afe Clone the repo in CI without GitHub-hosted actions
ci / vet-and-test (pull_request) Successful in 17s
Details
ci / docker-checkout-hostgateway (pull_request) Successful in 1s
Details 
Why: The runner cannot reach github.com, so cloning actions/checkout timed out before any test ran.
Evidence: mirrors.aliyun.com/golang/go1.27.1.linux-amd64.tar.gz returned HTTP 200. The workflow now fetches GITHUB_SHA with the job token and installs Go from that mirror.
Impact: Submodule checkout still requires SUBMODULE_TOKEN. A missing secret fails the job with that message.
Refs: feature/agent-stack-r0-r1
Co-authored-by: Cursor <cursoragent@cursor.com>
 2026-10-03 10:31:41 +00:00
weichenandCursor 7f7879b06b Fetch CI submodules over HTTPS and run the race detector
ci / vet-and-test (pull_request) Failing after 31s
Details
ci / docker-checkout-hostgateway (pull_request) Failing after 30s
Details 
Why: The runner has no SSH key, so recursive checkout failed, and tests had no module proxy or race detector.
Evidence: Workflow matches PROJECT.md vet, test, and race commands, with setup-go 1.27 and GOPROXY=https://goproxy.cn,direct.
Impact: vet-and-test runs on ubuntu-latest. SUBMODULE_TOKEN must be set by the owner before checkout can succeed.
Refs: feature/agent-stack-r0-r1
Co-authored-by: Cursor <cursoragent@cursor.com>
 2026-10-03 10:19:52 +00:00
weichenandCursor 3130190acb Reject invalid classifier refinements and scope slot validators
ci / vet-and-test (pull_request) Failing after 2s
Details
ci / docker-checkout-hostgateway (pull_request) Successful in 2s
Details 
Why: Refine could swap handlers or emit out-of-range scores, Ambiguous tops skipped slot extraction, and the process-global validator map coupled independent registries.
Evidence: GOWORK=off go vet ./... && go test -count=1 ./... && go test -race ./... passed, including TestRecognizeClassifierRejectsBadRefinement, TestRecognizeClassifierRebuildsTamperedHandler, TestRecognizeClassifierRefinedTopGetsSlots, and TestScopedSlotValidatorsDoNotInterfere. error-code-system and logging-system advanced to a3ee67b and 49b5d8b (docs-only).
Impact: illegal Refine results stay Ambiguous; a resolved Top is slotted; Options.SlotValidators wins over the deprecated global table.
Refs: I1, I2, I6
Co-authored-by: Cursor <cursoragent@cursor.com>
 2026-10-03 00:47:16 +00:00