Skip to content

Prevent repeated workspace show loops#460

Merged
chubes4 merged 1 commit into
mainfrom
fix-workspace-show-duplicate-loop
May 28, 2026
Merged

Prevent repeated workspace show loops#460
chubes4 merged 1 commit into
mainfrom
fix-workspace-show-duplicate-loop

Conversation

@chubes4

@chubes4 chubes4 commented May 28, 2026

Copy link
Copy Markdown
Member

Summary

  • stop marking workspace_show as repeatable so exact duplicate calls can be rejected by Data Machine
  • keep workspace_ls repeatable for intentional repeated directory inspection
  • cover the runtime duplicate policy in the workspace policy smoke test

Testing

  • php tests/smoke-workspace-policy-tools.php

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (openai/gpt-5.5)
  • Used for: Diagnosed the World Creator loop, drafted the workspace tool policy fix, and ran the targeted smoke test; Chris remains responsible for review and merge.

@homeboy-ci

homeboy-ci Bot commented May 28, 2026

Copy link
Copy Markdown
Contributor

Homeboy Results — data-machine-code

Lint

lint — passed

ℹ️ Full options: homeboy docs commands/lint
ℹ️ Save lint baseline: homeboy lint data-machine-code --baseline
Deep dive: homeboy lint data-machine-code --changed-since 9b9c76c

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-code-lint-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-code-lint-quality-Linux-node24 contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/data-machine-code/actions/runs/26594506180

Test

test — passed

ℹ️ Set --setting require_phpunit_tests=true when this component is expected to contain PHPUnit tests.
ℹ️ Auto-fix lint issues: homeboy refactor data-machine-code --from lint --write
ℹ️ Collect coverage: homeboy test data-machine-code --coverage
ℹ️ Pass args to test runner: homeboy test -- [args]
ℹ️ Full options: homeboy docs commands/test
Deep dive: homeboy test data-machine-code --changed-since 9b9c76c

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-code-test-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-code-test-quality-Linux-node24 contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/data-machine-code/actions/runs/26594506180

Audit

audit — passed

  • dead_code — 1 finding(s)
  • duplication — 1 finding(s)
  • repeated_literal_shape — 1 finding(s)
  • Total: 3 finding(s)

Deep dive: homeboy audit data-machine-code --changed-since 9b9c76c

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-code-audit-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-code-audit-quality-Linux-node24 contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/data-machine-code/actions/runs/26594506180
Tooling versions
  • Homeboy CLI: homeboy 0.201.0+7645358
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: 4062165d
  • Action: unknown@unknown

@chubes4
chubes4 merged commit a0ab632 into main May 28, 2026
9 of 10 checks passed
@chubes4
chubes4 deleted the fix-workspace-show-duplicate-loop branch May 28, 2026 19:20
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