CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

cell-architecture

Reference and active migration guide for Sentry's cell architecture. Explains what cells and localities are and why they're different, how requests reach cells via Synapse API routing, ingestion routing, and the control silo gateway, and how to safely query cross-cell data without silently missing results. The migration section covers how to do migration work: draining the URL_NAME_TO_ACTION registry in test_urls.py to zero (with a recipe for each action type), rolling deploy safety and the two-phase pattern required by independent sentry/getsentry deploys, and the region -> cell rename including what not to rename (DB columns, AWS refs, uptime regions, billing address). Also documents known issues with proposed fixes: integration TeamLinkageView routing, Jira cross-cell fan-out, and relocation endpoint routing.

67

Quality

84%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

SKILL.md
Quality
Evals
Security

Quality

Content

86%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

An exceptionally dense, high-signal reference: every section delivers concrete, executable, repo-specific guidance with no padding, and risky migration procedures carry verification steps. The main structural weakness is the absence of any bundle files — migration and known-issue material that belongs in separate reference files is inlined in an already long SKILL.md.

Suggestions

Split the Migration Guide (URL_NAME_TO_ACTION actions, rolling deploy safety, region -> cell rename) and/or Known Issues into references/ files, leaving SKILL.md as an architecture overview with clearly signaled one-level-deep links — the Status note already anticipates removing migration content, making the split natural.

Add explicit validation checkpoints to the migration recipes, e.g. "after removing an entry, run the test_urls.py check locally to confirm the registry is one step closer to zero".

Move the long TO_BE_DEPRECATED deletion checklist into its own subsection or reference, since it is the most step-heavy and destructive procedure.

DimensionReasoningScore

Conciseness

The body is dense with repo-specific facts Claude cannot know (exact files, decorators, env vars, ports like `:13000`/`:7899`, metric and option key names) and never explains general concepts Claude already knows (what Kafka, API gateways, or webhooks are). It matches the 5 anchor — lean, assumes competence, every token earns its place — rather than the 4 anchor, which requires over-explanation instances that could be trimmed.

5 / 5

Actionability

Guidance is fully executable and copy-paste ready: `devservices up --mode cell-routing`, `bin/send-cell-test-event.py` with PROJECT_ID/TARGET variants, the `org.locality.to_url(...)` snippet, the URL pattern `^api/0/_admin/cells/(?P<cell_id>[^/]+)/`, the `db_column="old_name"` rename trick, and `alias=` on `@instrumented_task`. Specific examples cover the common migration cases, matching the 5 anchor.

5 / 5

Workflow Clarity

Multi-step processes are clearly sequenced: the two-phase deploy pattern, the numbered three-step TeamLinkageView fix, and the TO_BE_DEPRECATED recipe which includes a verification step ("search the getsentry GitHub org for the URL name and path to find all callers") plus a stated CI enforcement mechanism. It falls just short of the 5 anchor because most recipes lack an explicit validate-before-proceed checkpoint (e.g. "run the test_urls.py check to verify the registry entry is gone"), though the destructive operations do include verification, so the cap at 3 does not apply.

4 / 5

Progressive Disclosure

There are no bundle files at all (references/, scripts/, assets/ are absent), so the entire ~280-line skill is one inlined document, with ~110 lines of Migration Guide and Known Issues that the rubric's good examples would place in separate reference files. Sections are well-organized with internal anchors and cross-skill pointers (hybrid-cloud-rpc, hybrid-cloud-outboxes), matching the 3 anchor (some structure, content that should be separate is inline) rather than the 4 anchor's "most content is appropriately placed".

3 / 5

Total

17

/

20

Passed

Description

75%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

A highly specific, distinctive description that comprehensively enumerates what the skill covers using concrete repo-level artifacts. Its main weakness is the absence of an explicit "Use when..." trigger clause, which caps completeness and leaves invocation conditions implied rather than stated.

Suggestions

Add an explicit trigger clause, e.g. "Use when working on Sentry cell/locality architecture, cross-cell data access, or the region -> cell migration in the sentry/getsentry codebases."

Include common synonym terms users might naturally say, such as "multi-region", "silo", or "monolith", to broaden natural trigger coverage.

Lead with the trigger guidance or append it as a final sentence so the "when" is as prominent as the detailed "what".

DimensionReasoningScore

Specificity

The description lists multiple concrete, specific items — "draining the URL_NAME_TO_ACTION registry in test_urls.py to zero", "the two-phase pattern required by independent sentry/getsentry deploys", and "what not to rename (DB columns, AWS refs, uptime regions, billing address)" — with comprehensive coverage of the skill's scope. It matches the 5 anchor (multiple specific concrete actions, comprehensive coverage) rather than the 4 anchor, which requires minor gaps in coverage; none are evident.

5 / 5

Completeness

The "what" is clearly and thoroughly answered (explains cells/localities, routing paths, migration procedures, known issues), but there is no "Use when..." clause or equivalent explicit trigger guidance — the judging guidelines state a missing explicit trigger clause caps completeness at 3. It clearly matches the 3 anchor (clear what, when missing or only weakly implied by "Reference and active migration guide").

3 / 5

Trigger Term Quality

Good natural keyword coverage: "cell architecture", "migration", "region -> cell rename", "cross-cell", "Synapse", and "control silo gateway" are phrases a Sentry developer would naturally say. A few common variations are missing (e.g. "multi-region", "silo", "monolith"), so it sits between the 4 anchor (good coverage, a few natural terms missing) and the 5 anchor (comprehensive with synonyms) — closer to 4.

4 / 5

Distinctiveness Conflict Risk

The description is tightly scoped to Sentry's cell architecture with unique identifiers ("URL_NAME_TO_ACTION registry in test_urls.py", "TeamLinkageView", "Jira cross-cell fan-out"), giving it a clear niche with distinct triggers and minimal conflict risk with other skills. It matches the 5 anchor exactly; the 4 anchor's "minor overlap risk" does not apply.

5 / 5

Total

17

/

20

Passed

Validation

100%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

getsentry/sentry

Path

.agents/skills/cell-architecture/SKILL.md

Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.