README.md:182 imports Arcp\Messages\Telemetry\LogRecord and Arcp\Messages\Telemetry\Metric, but the implementation registers LogEvent and MetricEvent in src/Envelope/MessageCatalog.php:163. The README then checks instanceof LogRecord and instanceof Metric, so the example cannot run. The same section lists event type names such as tool_call, tool_result, artifact_ref, and result_chunk, while the actual catalog uses dot-separated names such as tool.invoke, tool.result, artifact.ref, and job.result_chunk. samples/submit-and-stream/main.php:12 also prints stale log.event and metric.event names, and its metric payload omits the required unit.
These stale snippets are in the front-door documentation and teach users class names and wire names that do not exist in the current PHP SDK. They also obscure the actual telemetry API exposed by JobContext::emitLog() and JobContext::emitMetric().
Fix prompt: Update the README and affected sample to use LogEvent, MetricEvent, and the current catalog type names. Replace the stale event-name list with names produced by MessageType::typeName(), make the metric example include a non-empty unit, and add a lightweight documentation or sample smoke test that catches nonexistent PHP classes in README snippets.
README.md:182importsArcp\Messages\Telemetry\LogRecordandArcp\Messages\Telemetry\Metric, but the implementation registersLogEventandMetricEventinsrc/Envelope/MessageCatalog.php:163. The README then checksinstanceof LogRecordandinstanceof Metric, so the example cannot run. The same section lists event type names such astool_call,tool_result,artifact_ref, andresult_chunk, while the actual catalog uses dot-separated names such astool.invoke,tool.result,artifact.ref, andjob.result_chunk.samples/submit-and-stream/main.php:12also prints stalelog.eventandmetric.eventnames, and its metric payload omits the requiredunit.These stale snippets are in the front-door documentation and teach users class names and wire names that do not exist in the current PHP SDK. They also obscure the actual telemetry API exposed by
JobContext::emitLog()andJobContext::emitMetric().Fix prompt: Update the README and affected sample to use
LogEvent,MetricEvent, and the current catalog type names. Replace the stale event-name list with names produced byMessageType::typeName(), make the metric example include a non-empty unit, and add a lightweight documentation or sample smoke test that catches nonexistent PHP classes in README snippets.