Skip to content

[WIP] Refactor to create shared/serena-otlp.md bundle for workflows#42944

Closed
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/refactoring-create-shared-serena-otlp
Closed

[WIP] Refactor to create shared/serena-otlp.md bundle for workflows#42944
pelikhan with Copilot wants to merge 1 commit into
mainfrom
copilot/refactoring-create-shared-serena-otlp

Conversation

Copilot AI commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.


This section details on the original issue you should resolve

<issue_title>[refactoring] [skill] Create shared/serena-otlp.md bundle for 8 workflows using serena-go + otlp only</issue_title>
<issue_description>### Overview

8 workflows import exactly shared/mcp/serena-go.md + shared/otlp.md with no other shared components. These workflows all perform Go code analysis with semantic LSP tooling and all need telemetry. A shared/serena-otlp.md bundle (following the reporting-otlp.md pattern) would collapse two imports into one for this stable pairing.

Priority

Medium — 8 workflows affected, near-zero extraction complexity, stable pairing

Affected Workflows (8)

  • archie.md
  • daily-function-namer.md
  • developer-docs-consolidator.md
  • glossary-maintainer.md
  • linter-miner.md
  • sergo.md
  • terminal-stylist.md
  • typist.md

Estimated Impact

  • 8 workflows reduced from 2 imports to 1
  • 8 import lines removed
  • Codifies the natural coupling: any workflow doing semantic Go analysis should emit telemetry
  • New Go-analysis workflows get OTLP for free by importing a single component

Proposed Shared Component

Create .github/workflows/shared/serena-otlp.md:

---
# Serena Go code analysis + OTLP telemetry bundle
# Use for workflows that perform semantic Go code analysis and need telemetry.
#
# Usage:
#   imports:
#     - shared/serena-otlp.md

imports:
  - shared/mcp/serena-go.md
  - shared/otlp.md
---

Migration Plan

  1. Create shared/serena-otlp.md with the content above.
  2. In each of the 8 affected workflows, replace:
    imports:
      - shared/mcp/serena-go.md
      - shared/otlp.md
    with:
    imports:
      - shared/serena-otlp.md
  3. Run make recompile and verify compiled output is unchanged.

Example Usage

imports:
  - shared/serena-otlp.md
  # add shared/reporting.md or shared/reporting-otlp.md if publishing reports

References

  • Pattern precedent: .github/workflows/shared/reporting-otlp.md
  • Serena Go component: .github/workflows/shared/mcp/serena-go.md
  • OTLP component: .github/workflows/shared/otlp.md
  • Note: cloclo.md uses serena-go + reporting + otlp; it should migrate to serena-otlp.md + shared/reporting.md

[!WARNING]

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • awmgmcpg

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "awmgmcpg"

See Network Configuration for more information.

Generated by 🔍 Workflow Skill Extractor · 63.8 AIC · ⌖ 11.6 AIC · ⊞ 4.7K ·

  • expires on Jul 3, 2026, 9:03 PM UTC-08:00

Comments on the Issue (you are @copilot in this section)

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.

[refactoring] [skill] Create shared/serena-otlp.md bundle for 8 workflows using serena-go + otlp only

2 participants