docs(sitemap-promote): seed twitter + hackernews PoC at global path#1823
Merged
Conversation
Promote-prep commit — hackernews PoC v1.1 files copied from local overlay (~/.opencli/sites/hackernews/sitemap/) to the global seed path (skills/opencli-sitemap-author/references/site-memory/hackernews/sitemap/), with frontmatter source bumped local → global on all 10 files. @opencli-user's twitter v1.1 normalize will land on top of this branch, after which the joint promote PR opens against main. Files: - SITE.md, apis.md, pitfalls.md (site-level) - pages/front.md, feed.md, item.md, user.md - workflows/read-story.md, submit-story.md, upvote.md All Form B YAML actions, drop action-level verified_at/source, delimiter form `|`/`||`/`;`, adapter_health_update directives on adapter-primary actions, selector_pattern 5 types declared. Per v1.1 schema (#1822).
- Add 12 twitter sitemap files at skills/opencli-sitemap-author/references/site-memory/twitter/sitemap/ - frontmatter source bumped local -> global - v1.1 schema applied: Form B compact YAML actions, adapter_health_update directives on write workflows, regularized | / || / ; delimiters, selector_pattern anchors where applicable - pairs with hackernews seed in prior commit; both become global seed at promote
Align with twitter sitemap files in this PR and with the v1.1 schema applied (Form B YAML actions, adapter_health_update directives, etc.). Catch from review — twitter files already use schema_version: 1.1 throughout.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Promote Sitemap Hub PoC v1.1 from local-only to global seed for two complementary sites:
@opencli-user@opencli-质量官Both seeds land at
skills/opencli-sitemap-author/references/site-memory/{twitter,hackernews}/sitemap/and bump frontmattersource: local → global. The two PoCs together close the cross-validation loop on the v1.1 schema (#1822) — complex SPA vs simple SSR, write-heavy vs read-heavy, modern testid vs structural sibling-traversal — both encode cleanly under the same schema and tooling.v1.1 schema application checklist (per #1822)
Both PoC sets apply the 12-patch schema delta as a unit:
pre / do / post / fail / recover / evidence, ~80 tokens) used as default; Form A markdown reserved for actions that genuinely need prose explanation.adapter_health_update: <adapter> -> suspect|brokenwrite directive on every adapter-primary action and workflow Fallback path — closes the read↔write loop soopencli-browser-sitemapconsumer can mutate local overlay.|for failure-signal enums,||for fallback priority indo:,;for sequential recovery steps.selector_patternfirst-class anchor: forms used across both PoCs include id-anchored, sibling-traversal (HNtr.athing[id="<id>"] + tr a[href^="item?id="]), data-testid (Twitter[data-testid="like"]), form-name, ARIA role. Discouraged forms (nth-child(<rank>)/ single class / text-only) avoided._prefix + emptyurl_patterns: []) used for cross-page UI primitives — Twitter_tweet_card.mdreferences fromhome / profile / status.Cross-PoC token sizes (per file, bytes ≈ token×1.5 for English-heavy content)
Per-file all under the 800-token hard limit. Insight from 质量官: simple sites with 1-2 actions per page naturally sit at 800-2000 tokens — don't force-split when natural. Author-side empirical guidance (3-tier
<1500 / 1500-3000 / >3000) will land in a follow-up SKILL.md patch.Trust-reality verification
Both seeds were verified end-to-end against live sites before promotion:
pitfalls.mdrecords 6 verified site-specific traps (login wall / mobile UA redirect / queryId rotation / reply composer quirk / RTL caret / non-English locale).read-story / submit-story / upvote) tested acrossnews / newest / itemroutes;pitfalls.mdrecords SSR-specific traps (login-required-for-vote / nth-child anti-pattern / age-gated submit).What's NOT in scope (deferred)
opencli-browser-sitemapskill to land Phase 2 features.Test plan
@opencli-质量官(review by author of hackernews seed)source: globalon all 22 filesadapter_health_updatedirective presencecc
@opencli-质量官(review owner per thread)