docs: v3.5.39 release-readiness fixes (course drift, tutor scope, stale checklist/TODO)#158
Merged
Merged
Conversation
ko/ja STORE_LISTING headers were bumped to "18 courses" in #157 but their course enumeration still listed only 17 — "AI Fluency for Small Businesses" (the course #157 actually added) was missing from both localized lists. Also corrects three stale "v3.5.34" references in RELEASE_CHECKLIST to v3.5.39 (the shipping version), including the SNS-launch gate that told us to wait for the listing to reflect a version that will never be uploaded.
Second-pass /code-review findings on the v3.5.39 release-readiness docs: - README: AI Tutor section + intro now state the tutor/Puter bridge run on anthropic.skilljar.com only; other detected Skilljar AI tenants get dictionary + Google Translate but no tutor (matches the v3.5.39 host gate). - TODO.md: mark the shipped learning-companion items (bookmarks, resume, Tools-menu overlay, TOC) and the small-businesses course wiring done; refresh the stale v3.5.36 header/date. - RELEASE_CHECKLIST: the raw `skillbridge.zip` fallback is gitignored and not rebuilt by build:bundle:zip, so it silently lagged at 3.5.38 while the bundle was 3.5.39. Note it must be regenerated with `npm run build:zip` immediately before use.
heznpc
changed the title
… disclosure The CWS submit was blocked by "개인정보처리방침 링크에 연결할 수 없습니다": the dashboard had the lowercase github.io path, which 404s (GitHub Pages repo paths are case-sensitive; capital-B `skillBridge` returns 200). Replace the misleading "/privacy (lowercase) verified 200" note with the case-sensitivity warning. Add a Privacy-tab subsection capturing the v1.0.1→v3.5.39 deltas that trigger re-review: remote code is now NO (Puter bundled as src/bridge/puter.js, loaded via chrome.runtime.getURL, no remote fallback), "Website content" data type must be checked (page text → Google Translate, lesson context → Puter/Gemini/Claude), and alarms / api.github.com need fresh justifications while activeTab/tabs drop off. Also flag that the live store-listing icon is still the old radial-spark and must be re-uploaded separately from the package.
heznpc
added a commit
that referenced
this pull request
Jun 1, 2026
Drop the hand-maintained ko/ja localized listings — every CWS locale falls back to the English default, and keeping three parallel free-form copies in sync was the source of the 17/18 course-count drift (#158) for marginal per-market gain. The extension UI stays localized in 11 languages; only the store metadata is now English-only. Reframe the EN listing away from the "certificate / Korea-first" POSITIONING lock toward broad global appeal + engineering craft: - Summary trimmed 135 -> 130 chars (was over the 132 limit) and reworded. - Description lead rewritten: 32 languages / "your language" instead of non-English-learner-certificate framing; leads with the curated-terminology + background-LLM-verification + in-page-tutor craft. - Header/What's-New bumped to v3.5.39. Clean up references in RELEASE_CHECKLIST (locale row) and TODO (drop the multilingual-listing items).
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
1 participant
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.
Two
/code-reviewpasses over the v3.5.39 release-readiness docs (follow-up to #157). Docs/markdown only — no code paths touched.Pass 1
AI Fluency for Small Businesses) to the EN enumeration, but left ko/ja at 17. Adds the missing course to both localized lists.v3.5.34refs inRELEASE_CHECKLIST.md(incl. the SNS-launch gate "wait until the listing reflects v3.5.34") corrected to v3.5.39; line 21 kept as thev3.5.34–3.5.39range.Pass 2
store-assets/skillbridge.zipis gitignored and not rebuilt bybuild:bundle:zip, so it lagged at manifest 3.5.38 while the bundle was 3.5.39. Using it as fallback would ship the pre-fix (unscoped) bridge. Checklist now flags itnpm run build:zipimmediately before use; the local copy was regenerated to 3.5.39.anthropic.skilljar.com, but README implied the tutor runs on any detected Skilljar AI tenant. Intro + AI Tutor section now state: other tenants get dictionary + Google Translate only; the AI Tutor is Anthropic-Academy-exclusive.ai-fluency-for-small-businessesas TODO thoughcheck:academypasses. Marked done with notes; refreshed header/date.Verify
STORE_LISTING*.mdenumerate 18 courses (small-businesses present)check:academy+check:dict-coveragepass;format:checkclean (markdown is outside the prettier scope, so no churn introduced)