test(safewrite): add symlink-parent and broken-symlink edge-case tests #424
Patel230ci.yml
on: push
format
21s
vet
1m 15s
module hygiene
1m 16s
secrets
13s
markdown
14s
deadcode
23s
duplication
15s
smoke
1m 48s
Matrix: build
Annotations
7 errors and 2 warnings
|
module hygiene
Process completed with exit code 1.
|
|
module hygiene
go.mod / go.sum / go.work files out of date — run 'go work sync' locally and commit
|
|
lint
Process completed with exit code 1.
|
|
lint:
internal/jsonc/validate.go#L62
var knownFields is unused (unused)
|
|
lint:
internal/permissions/verdict.go#L128
S1002: should omit comparison to bool constant, can be simplified to !v.Allowed (staticcheck)
|
|
lint:
internal/session/session_gain.go#L198
Error return value of `rows.Close` is not checked (errcheck)
|
|
security
Process completed with exit code 1.
|
|
duplication
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/setup-node@v4. Actions will be forced to run with Node.js 24 by default starting June 16th, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
test (race + coverage)
Found t.Skip() without tracking issue:\n./cmd/clipboard_test.go:18: t.Skip() without tracking issue\n./cmd/clipboard_test.go:23: t.Skip() without tracking issue\n./cmd/dx_test.go:326: t.Skip() without tracking issue\n./cmd/chat_model_test.go:137: t.Skip() without tracking issue\n./cmd/chat_model_test.go:211: t.Skip() without tracking issue\n./cmd/path_test.go:13: t.Skip() without tracking issue\n./internal/tool/safety_test.go:291: t.Skip() without tracking issue\n./internal/tool/safety_test.go:300: t.Skip() without tracking issue\n./internal/tool/path_resolver_test.go:357: t.Skip() without tracking issue\n./internal/plugin/manager_test.go:164: t.Skip() without tracking issue\n./internal/plugin/manager_test.go:204: t.Skip() without tracking issue\n./internal/plugin/manager_test.go:244: t.Skip() without tracking issue\n./internal/plugin/manager_test.go:509: t.Skip() without tracking issue\n./internal/plugin/manager_test.go:580: t.Skip() without tracking issue\n./internal/plugin/manager_test.go:725: t.Skip() without tracking issue\n./internal/plugin/manager_test.go:763: t.Skip() without tracking issue\n./internal/plugin/bridge_test.go:39: t.Skip() without tracking issue\n./internal/feature/shellmode/shellmode_test.go:55: t.Skip() without tracking issue\n./internal/feature/shellmode/shellmode_test.go:74: t.Skip() without tracking issue\n./internal/feature/shellmode/shellmode_test.go:87: t.Skip() without tracking issue\n./internal/feature/shellmode/shellmode_test.go:100: t.Skip() without tracking issue\n./internal/feature/shellmode/shellmode_test.go:116: t.Skip() without tracking issue\n./internal/permissions/boundary_test.go:116: t.Skip() without tracking issue\n./internal/permissions/boundary_test.go:577: t.Skip() without tracking issue\n./internal/multiagent/parallel/parallel_test.go:102: t.Skip() without tracking issue\n./internal/config/catalog_startup_robust_test.go:18: t.Skip() without tracking issue\n./internal/config/setup_status_test.go:82: t.Skip() without tracking issue\n./internal/config/catalog_startup_test.go:75: t.Skip() without tracking issue\n./internal/config/catalog_startup_test.go:92: t.Skip() without tracking issue\n./internal/config/catalog_startup_test.go:115: t.Skip() without tracking issue\n./internal/config/catalog_startup_test.go:128: t.Skip() without tracking issue\n./internal/config/catalog_startup_test.go:143: t.Skip() without tracking issue\n./internal/sandbox/landlock_test.go:68: t.Skip() without tracking issue\n./internal/sandbox/landlock_test.go:104: t.Skip() without tracking issue\n./internal/prompts/workspace_test.go:23: t.Skip() without tracking issue\n./internal/prompts/workspace_test.go:97: t.Skip() without tracking issue\n./internal/prompts/workspace_test.go:120: t.Skip() without tracking issue\n./internal/prompt/suggestion_test.go:65: t.Skip() without tracking issue\n./internal/intelligence/memory/autodream_test.go:87: t.Skip() without tracking issue\n./internal/intelligence/memory/yaad_bridge_integration_test.go:24: t.Skip() without tracking issue\n./internal/intelligence/memory/yaad_bridge_integration_test.go:31: t.Skip() without tracking issue\n./internal/intelligence/memory/yaad_bridge_integration_test.go:42: t.Skip() without tracking issue\n./internal/intelligence/memory/yaad_bridge_integration_test.go:56: t.Skip() without tracking issue\n./internal/intelligence/memory/yaad_bridge_integration_test.go:64: t.Skip() without tracking issue\n./internal/intelligence/memory/yaad_bridge_integration_test.go:76: t.Skip() without tracking issue\n./internal/intelligence/memory/yaad_bridge_integration_test.go:92: t.Skip() without tracking issue\n./internal/provider/routing/tiers_test.go:30: t.Skip() without tracking issue\n./internal/engine/cost/cost_optimizer_test.go:255: t.Skip() without tracking issue\n./internal/engine/validate_test.go:55: t.Skip() without tracking issue\n./internal/engine/validate_test.go:71: t.Skip() without tracking issue\n./internal/engine/review/review_bot_test.go:284: t.Skip() without tracking issue\n./internal/engine/validation/lint_loop_test.go:306: t.Skip() witho
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
coverage-test
Expired
|
446 KB |
sha256:f064c51d31fa36d7f37373bc8e27903c110a7e4369785e69a035729aa1863b5f
|
|