Skip to content

chore(doc): rename Studio doc to Observability & Studio with OpenTelemetry tracing guide#1186

Open
LearningGp wants to merge 1 commit intoagentscope-ai:mainfrom
LearningGp:fix/doc-ot
Open

chore(doc): rename Studio doc to Observability & Studio with OpenTelemetry tracing guide#1186
LearningGp wants to merge 1 commit intoagentscope-ai:mainfrom
LearningGp:fix/doc-ot

Conversation

@LearningGp
Copy link
Copy Markdown
Collaborator

AgentScope-Java Version

1.0.11

Description

Replace studio.md with observability.md covering both Studio visual debugging (unchanged) and new OpenTelemetry tracing configuration guide for Langfuse and other OTLP backends.

Checklist

Please check the following items before code is ready to be reviewed.

  • Code has been formatted with mvn spotless:apply
  • All tests are passing (mvn test)
  • Javadoc comments are complete and follow project conventions
  • Related documentation has been updated (e.g. links, examples, etc.)
  • Code is ready for review

…metry tracing guide

Replace studio.md with observability.md covering both Studio visual
debugging (unchanged) and new OpenTelemetry tracing configuration
guide for Langfuse and other OTLP backends.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Renames and expands the Studio documentation into a consolidated “Observability & Studio” guide, adding OpenTelemetry (OTLP) tracing setup alongside the existing Studio visual debugging content.

Changes:

  • Updated EN/ZH observability docs to include a combined Studio + OpenTelemetry tracing guide.
  • Added TelemetryTracer + TracerRegistry usage examples, including Langfuse OTLP configuration.
  • Updated the docs TOC to point from task/studio to task/observability in both EN and ZH.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
docs/zh/task/observability.md Adds OpenTelemetry tracing section and restructures Studio content under a unified observability page (ZH).
docs/en/task/observability.md Adds OpenTelemetry tracing section and restructures Studio content under a unified observability page (EN).
docs/_toc.yml Replaces Studio TOC entries with Observability & Studio entries for EN/ZH navigation.
Comments suppressed due to low confidence (4)

docs/en/task/observability.md:321

  • The Jaeger example uses http://localhost:4317, but TelemetryTracer is implemented with OtlpHttpSpanExporter (OTLP/HTTP), which expects an HTTP endpoint (commonly http://localhost:4318/v1/traces). As written, this endpoint is likely to fail or confuse users; please update the example (and wording) to match OTLP/HTTP expectations.
    docs/zh/task/observability.md:319
  • 这里描述为“支持 OTLP 协议的平台都可以使用”,但 TelemetryTracer 实际是基于 OtlpHttpSpanExporter(OTLP/HTTP)。建议在文档中明确需要 OTLP/HTTP 端点(通常形如 http://host:4318/v1/traces),避免用户按 gRPC(4317) 配置导致无法上报。
    docs/en/task/observability.md:246
  • This section says traces can be exported to “any platform that supports the OTLP protocol”, but TelemetryTracer uses OtlpHttpSpanExporter (OTLP/HTTP). Please clarify that an OTLP/HTTP endpoint (typically including /v1/traces) is required, so readers don’t try OTLP/gRPC-only endpoints.
    docs/zh/task/observability.md:244
  • 此处说明可导出到“任何支持 OTLP 协议的平台”,但 TelemetryTracer 使用的是 OTLP/HTTP 导出器。建议明确需要 OTLP/HTTP 的 endpoint(通常包含 /v1/traces),以免读者误配到仅支持 OTLP/gRPC 的地址。

Comment on lines 56 to 60
title: Structured Output
- file: en/task/studio
title: Studio
- file: en/task/observability
title: Observability & Studio
- file: en/task/agui
title: AG-UI Protocol
Copy link

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The TOC now points to en/task/observability, but there are still docs linking to the removed task/studio.md (e.g., docs/en/intro.md). Please update those links so users don’t hit a 404.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot apply changes based on this feedback

Comment on lines 131 to 136
title: 结构化输出
- file: zh/task/studio
title: Studio
- file: zh/task/observability
title: 可观测与调试
- file: zh/task/agui
title: AG-UI 协议
- file: zh/task/a2a
Copy link

Copilot AI Apr 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

目录已从 zh/task/studio 改为 zh/task/observability,但仍有文档链接指向已移除的 task/studio.md(如 docs/zh/intro.md)。请同步更新这些链接,避免导航到不存在的页面。

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot apply changes based on this feedback

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copy link
Copy Markdown
Collaborator

@chickenlj chickenlj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

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.

3 participants