Refactor PHP codebase to use core/go wrappers and clear findings

.gitignore

@@ -31,3 +31,6 @@ infection-summary.log
 # Worktrees
 .worktrees/
 .core/
+
+# committed build output — keep out of git
+/bin/core-php-cgo

go/go.mod

@@ -3,7 +3,7 @@ module dappco.re/go/php
 go 1.26.2
 
 require (
-	dappco.re/go v0.9.0
+	dappco.re/go v0.10.4
 	dappco.re/go/cli v0.9.0
 	dappco.re/go/i18n v0.9.0
 	dappco.re/go/io v0.9.0

go/go.sum

@@ -1,5 +1,9 @@
 dappco.re/go v0.9.0 h1:4ruZRNqKDDva8o6g65tYggjGVe42E6/lMZfVKXtr3p0=
 dappco.re/go v0.9.0/go.mod h1:xapr7fLK4/9Pu2iSCr4qZuIuatmtx1j56zS/oPDbGyQ=
+dappco.re/go v0.10.3 h1:aViRNxdg2jG84P6RsiD+aSta+GcFJwGXMNQPjFPbJ9g=
+dappco.re/go v0.10.3/go.mod h1:xapr7fLK4/9Pu2iSCr4qZuIuatmtx1j56zS/oPDbGyQ=
+dappco.re/go v0.10.4 h1:vir5AK8AkHbTxhPUT0et6Tc0P8i/i+gLInM0LRLt1EU=
+dappco.re/go v0.10.4/go.mod h1:xapr7fLK4/9Pu2iSCr4qZuIuatmtx1j56zS/oPDbGyQ=
 dappco.re/go/cli v0.9.0 h1:KY8V75vqi4HJtZwWEpY8QZT6ukpNJ4FSatphSOBmBJ8=
 dappco.re/go/cli v0.9.0/go.mod h1:6PQIZtv319UKowolKG8tUIRdcZ6nkbFsRe+ZJi8KiQ4=
 dappco.re/go/i18n v0.9.0 h1:ZZuFSBzCGYrQotXs/tOxxiPLB+NkhjYF+HrxYkWK6+U=

go/pkg/php/bridge.go

@@ -2,11 +2,12 @@ package php
 
 import (
 	"context"
-	`encoding/json`
-	`fmt`
-	`log`
+	"encoding/json"
+	"io"
 	"net"
 	"net/http"
+
+	core "dappco.re/go"
 )
 
 // BridgeHandler is the interface that the host application implements to
@@ -45,10 +46,15 @@ func NewBridge(handler BridgeHandler) (*Bridge, error) { // Result boundary
 			Args   json.RawMessage `json:"args"`
 		}
 		r.Body = http.MaxBytesReader(w, r.Body, 1<<20)
-		if err := json.NewDecoder(r.Body).Decode(&req); err != nil {
+		body, err := io.ReadAll(r.Body)
+		if err != nil {
 			http.Error(w, err.Error(), http.StatusBadRequest)
 			return
 		}
+		if r := core.JSONUnmarshal(body, &req); !r.OK {
+			http.Error(w, r.Error(), http.StatusBadRequest)
+			return
+		}
 
 		result, err := handler.HandleBridgeCall(req.Method, req.Args)
 		if err != nil {
@@ -61,19 +67,19 @@ func NewBridge(handler BridgeHandler) (*Bridge, error) { // Result boundary
 	// Listen on a random available port (localhost only)
 	listener, err := net.Listen("tcp", "127.0.0.1:0")
 	if err != nil {
-		return nil, fmt.Errorf("listen: %w", err)
+		return nil, core.E("php.NewBridge", "listen", err)
 	}
 
 	bridge.port = listener.Addr().(*net.TCPAddr).Port
 	bridge.server = &http.Server{Handler: mux}
 
 	go func() {
 		if err := bridge.server.Serve(listener); err != nil && err != http.ErrServerClosed {
-			log.Printf("go-php: bridge error: %v", err)
+			core.Println(core.Sprintf("go-php: bridge error: %v", err))
 		}
 	}()
 
-	log.Printf("go-php: bridge listening on http://127.0.0.1:%d", bridge.port)
+	core.Println(core.Sprintf("go-php: bridge listening on http://127.0.0.1:%d", bridge.port))
 	return bridge, nil
 }
 
@@ -84,7 +90,7 @@ func (b *Bridge) Port() int {
 
 // URL returns the full base URL of the bridge.
 func (b *Bridge) URL() string {
-	return fmt.Sprintf("http://127.0.0.1:%d", b.port)
+	return core.Sprintf("http://127.0.0.1:%d", b.port)
 }
 
 // Shutdown gracefully stops the bridge server.
@@ -94,7 +100,12 @@ func (b *Bridge) Shutdown(ctx context.Context) error { // Result boundary
 
 func bridgeJSON(w http.ResponseWriter, v any) {
 	w.Header().Set("Content-Type", "application/json")
-	if err := json.NewEncoder(w).Encode(v); err != nil {
-		log.Printf("go-php: bridge encode error: %v", err)
+	r := core.JSONMarshal(v)
+	if !r.OK {
+		core.Println(core.Sprintf("go-php: bridge encode error: %v", r.Error()))
+		return
+	}
+	if _, err := w.Write(r.Value.([]byte)); err != nil {
+		core.Println(core.Sprintf("go-php: bridge write error: %v", err))
 	}
 }

go/pkg/php/cmd.go

@@ -1,9 +1,6 @@
 package php
 
 import (
-	`os`
-	`path/filepath`
-
 	core "dappco.re/go"
 	"dappco.re/go/cli/pkg/cli"
 	"dappco.re/go/io"
@@ -115,7 +112,8 @@ func activateWorkspacePackage() error { // Result boundary
 		return nil
 	}
 
-	if err := os.Chdir(targetDir); err != nil {
+	if r := core.Chdir(targetDir); !r.OK {
+		err, _ := r.Value.(error)
 		return core.E("php", "failed to change directory to active package", err)
 	}
 
@@ -142,9 +140,9 @@ func activeWorkspacePackageDir(wsRoot string, config *workspaceConfig) string {
 	if pkgDir == "" {
 		pkgDir = "./packages"
 	}
-	if !filepath.IsAbs(pkgDir) {
-		pkgDir = filepath.Join(wsRoot, pkgDir)
+	if !core.PathIsAbs(pkgDir) {
+		pkgDir = core.PathJoin(wsRoot, pkgDir)
 	}
 
-	return filepath.Join(pkgDir, config.Active)
+	return core.PathJoin(pkgDir, config.Active)
 }