Skip to content

Add SSI import repair planning task#2383

Closed
chubes4 wants to merge 1 commit into
mainfrom
issue/2381-ssi-sandbox-repair
Closed

Add SSI import repair planning task#2383
chubes4 wants to merge 1 commit into
mainfrom
issue/2381-ssi-sandbox-repair

Conversation

@chubes4
Copy link
Copy Markdown
Member

@chubes4 chubes4 commented May 30, 2026

Summary

  • Adds a ssi_import_repair_plan system task for sandbox-local Static Site Importer diagnostics repair planning.
  • Parses fallback block, unresolved asset, broken link, and conversion diagnostics into a structured plan/summary without writing files or mutating the host site.
  • Validates source-tree scope and refuses destructive or out-of-root requested edits.

Closes #2381

Tests

  • php tests/ssi-import-repair-plan-smoke.php
  • ./vendor/bin/phpcs inc/Engine/AI/System/Tasks/SSIImportRepairPlanTask.php inc/Engine/AI/System/SystemAgentServiceProvider.php tests/ssi-import-repair-plan-smoke.php
  • php -l inc/Engine/AI/System/Tasks/SSIImportRepairPlanTask.php
  • php -l inc/Engine/AI/System/SystemAgentServiceProvider.php
  • php -l tests/ssi-import-repair-plan-smoke.php

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (openai/gpt-5.5)
  • Used for: Drafted the system task implementation, smoke coverage, and PR description; Chris remains responsible for review and testing.

@homeboy-ci
Copy link
Copy Markdown
Contributor

homeboy-ci Bot commented May 30, 2026

Homeboy Results — data-machine

Lint

lint — failed

ℹ️ Auto-fix: homeboy lint data-machine --path /home/runner/work/data-machine/data-machine --changed-since b95cb35 --fix (or homeboy refactor data-machine --path /home/runner/work/data-machine/data-machine --changed-since b95cb35 --from lint --write)
ℹ️ Some issues may require manual fixes
ℹ️ Full options: homeboy docs commands/lint
Deep dive: homeboy lint data-machine --changed-since b95cb35

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-lint-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-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/actions/runs/26694868596

Test

test — passed

ℹ️ Auto-fix lint issues: homeboy refactor data-machine --from lint --write
ℹ️ Collect coverage: homeboy test data-machine --coverage
ℹ️ Pass args to test runner: homeboy test -- [args]
ℹ️ Full options: homeboy docs commands/test
Deep dive: homeboy test data-machine --changed-since b95cb35

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-test-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-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/actions/runs/26694868596

Audit

audit — passed

  • requested_detectors — 11 finding(s)
  • intra-method-duplication — 9 finding(s)
  • dead_code — 3 finding(s)
  • Directives — 1 finding(s)
  • Retention — 1 finding(s)
  • Total: 25 finding(s)

Deep dive: homeboy audit data-machine --changed-since b95cb35

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-audit-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-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/actions/runs/26694868596
Tooling versions
  • Homeboy CLI: homeboy 0.210.0+4b2873bf
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: d46af358
  • Action: unknown@unknown

@chubes4
Copy link
Copy Markdown
Member Author

chubes4 commented May 30, 2026

Closing because this implements the wrong ownership boundary. Data Machine core must not know about Static Site Importer repair semantics; Studio Web should inject its own sandbox MU plugin/task into Codebox and use Data Machine only as a generic runtime ingredient.

@chubes4 chubes4 closed this May 30, 2026
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.

Add sandbox repair task for SSI import diagnostics

1 participant