Verify technical accuracy of JavaScript concept pages by checking code examples, MDN/ECMAScript compliance, and external resources to prevent misinformation
npxskills add leonardomso/33-js-concepts--skill fact-checkLoading…
Use this skill to verify the technical accuracy of concept documentation pages for the 33 JavaScript Concepts project. This ensures we're not spreading misinformation about JavaScript.
Follow these five phases in order for a complete fact check.
Every code example in the concept page must be verified for accuracy.
Identify all code blocks in the document
For each code block:
// "string")For "wrong" examples (marked with ❌):
For "correct" examples (marked with ✓):
Run project tests:
# Run all tests
npm test
# Run tests for a specific concept
npm test -- tests/fundamentals/call-stack/
npm test -- tests/fundamentals/primitive-types/
Check test coverage:
/tests/{category}/{concept-name}/| Check | How to Verify |
|---|---|
console.log outputs match comments | Run code or trace mentally |
| Variables are correctly named/used | Read through logic |
| Functions return expected values | Trace execution |
| Async code resolves in stated order | Understand event loop |
| Error examples actually throw | Test in try/catch |
| Array/object methods return correct types | Check MDN |
typeof results are accurate | Test common cases |
| Strict mode behavior noted if relevant | Check if example depends on it |
// Watch for these common mistakes:
// 1. typeof null
typeof null // "object" (not "null"!)
// 2. Array methods that return new arrays vs mutate
const arr = [1, 2, 3]
arr.push(4) // Returns 4 (length), not the array!
arr.map(x => x*2) // Returns NEW array, doesn't mutate
// 3. Promise resolution order
Promise.resolve().then(() => console.log('micro'))
setTimeout(() => console.log('macro'), 0)
console.log('sync')
// Output: sync, micro, macro (NOT sync, macro, micro)
// 4. Comparison results
[] == false // true
[] === false // false
![] // false (empty array is truthy!)
// 5. this binding
const obj = {
name: 'Alice',
greet: () => console.log(this.name) // undefined! Arrow has no this
}
All claims about JavaScript APIs, methods, and behavior should align with MDN documentation.
Check all MDN links:
Verify API descriptions:
Check for deprecated APIs:
Verify browser compatibility claims:
| Content Type | MDN URL Pattern |
|---|---|
| Web APIs | https://developer.mozilla.org/en-US/docs/Web/API/{APIName} |
| Global Objects | https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/{Object} |
| Statements | https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/{Statement} |
| Operators | https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/{Operator} |
| HTTP | https://developer.mozilla.org/en-US/docs/Web/HTTP |
| Claim Type | What to Check |
|---|---|
| Method signature | Parameters, optional params, return type |
| Return value | Exact type and possible values |
| Side effects | Does it mutate? What does it affect? |
| Exceptions | What errors can it throw? |
| Browser support | Compatibility tables |
| Deprecation status | Any deprecation warnings? |
For nuanced JavaScript behavior, verify against the ECMAScript specification.
The ECMAScript specification is at: https://tc39.es/ecma262/
| Concept | Spec Section |
|---|---|
| Type coercion | Abstract Operations (7.1) |
| Equality | Abstract Equality Comparison (7.2.14), Strict Equality (7.2.15) |
| typeof | The typeof Operator (13.5.3) |
| Objects | Ordinary and Exotic Objects' Behaviours (10) |
| Functions | ECMAScript Function Objects (10.2) |
| this binding | ResolveThisBinding (9.4.4) |
| Promises | Promise Objects (27.2) |
| Iteration | Iteration (27.1) |
// Claim: "typeof null returns 'object' due to a bug"
// Spec says: typeof null → "object" (Table 41)
// Historical context: This is a known quirk from JS 1.0
// Verdict: ✓ Correct, though calling it a "bug" is slightly informal
// Claim: "Promises always resolve asynchronously"
// Spec says: Promise reaction jobs are enqueued (27.2.1.3.2)
// Verdict: ✓ Correct - even resolved promises schedule microtasks
// Claim: "=== is faster than =="
// Spec says: Nothing about performance
// Verdict: ⚠️ Needs nuance - this is implementation-dependent
All external links (articles, videos, courses) must be verified.
Check link accessibility:
Verify content accuracy:
Check publication date:
Verify description accuracy:
| Check | Pass Criteria |
|---|---|
| Link works | Returns 200, content loads |
| Not paywalled | Free to access (or clearly marked) |
| JavaScript-focused | Not primarily about other languages |
| Not outdated | Post-2015 for modern JS topics |
| Accurate description | Our description matches actual content |
| No anti-patterns | Doesn't teach bad practices |
| Reputable source | From known/trusted creators |
var everywhere for ES6+ topicsReview all prose claims about JavaScript behavior.
| Claim Type | How to Verify |
|---|---|
| Performance claims | Need benchmarks or caveats |
| Browser behavior | Specify which browsers, check MDN |
| Historical claims | Verify dates/versions |
| "Always" or "never" statements | Check for exceptions |
| Comparisons (X vs Y) | Verify both sides accurately |
❌ "async/await is always better than Promises"
→ Verify: Not always - Promise.all() is better for parallel operations
❌ "JavaScript is an interpreted language"
→ Verify: Modern JS engines use JIT compilation
❌ "Objects are passed by reference"
→ Verify: Technically "passed by sharing" - the reference is passed by value
❌ "=== is faster than =="
→ Verify: Implementation-dependent, not guaranteed by spec
✓ "JavaScript is single-threaded"
→ Verify: Correct for the main thread (Web Workers are separate)
✓ "Promises always resolve asynchronously"
→ Verify: Correct per ECMAScript spec
Watch for these misconceptions being stated as fact.
| Misconception | Reality | How to Verify |
|---|---|---|
typeof null === "object" is intentional | It's a bug from JS 1.0 that can't be fixed for compatibility | Historical context, TC39 discussions |
| JavaScript has no types | JS is dynamically typed, not untyped | ECMAScript spec defines types |
== is always wrong | == null checks both null and undefined, has valid uses | Many style guides allow this pattern |
NaN === NaN is false "by mistake" | It's intentional per IEEE 754 floating point spec | IEEE 754 standard |
| Misconception | Reality | How to Verify |
|---|---|---|
| Arrow functions are just shorter syntax | They have no this, arguments, super, or new.target | MDN, ECMAScript spec |
var is hoisted to function scope with its value | Only declaration is hoisted, not initialization | Code test, MDN |
| Closures are a special opt-in feature | All functions in JS are closures | ECMAScript spec |
| IIFEs are obsolete | Still useful for one-time initialization | Modern codebases still use them |
| Misconception | Reality | How to Verify |
|---|---|---|
| Promises run in parallel | JS is single-threaded; Promises are async, not parallel | Event loop explanation |
async/await is different from Promises | It's syntactic sugar over Promises | MDN, can await any thenable |
setTimeout(fn, 0) runs immediately | Runs after current execution + microtasks | Event loop, code test |
await pauses the entire program | Only pauses the async function, not the event loop | Code test |
| Misconception | Reality | How to Verify |
|---|---|---|
| Objects are "passed by reference" | References are passed by value ("pass by sharing") | Reassignment test |
const makes objects immutable | const prevents reassignment, not mutation | Code test |
| Everything in JavaScript is an object | Primitives are not objects (though they have wrappers) | typeof tests, MDN |
Object.freeze() creates deep immutability | It's shallow - nested objects can still be mutated | Code test |
| Misconception | Reality | How to Verify |
|---|---|---|
=== is always faster than == | Implementation-dependent, not spec-guaranteed | Benchmarks vary |
for loops are faster than forEach | Modern engines optimize both; depends on use case | Benchmark |
| Arrow functions are faster | No performance difference, just different behavior | Benchmark |
| Avoiding DOM manipulation is always faster | Sometimes batch mutations are slower than individual | Depends on browser, use case |
Running the project's test suite is a key part of fact-checking.
# Run all tests
npm test
# Run tests in watch mode
npm run test:watch
# Run tests with coverage
npm run test:coverage
# Run tests for specific concept
npm test -- tests/fundamentals/call-stack/
npm test -- tests/fundamentals/primitive-types/
npm test -- tests/fundamentals/value-reference-types/
npm test -- tests/fundamentals/type-coercion/
npm test -- tests/fundamentals/equality-operators/
npm test -- tests/fundamentals/scope-and-closures/
tests/
├── fundamentals/ # Concepts 1-6
│ ├── call-stack/
│ ├── primitive-types/
│ ├── value-reference-types/
│ ├── type-coercion/
│ ├── equality-operators/
│ └── scope-and-closures/
├── functions-execution/ # Concepts 7-8
│ ├── event-loop/
│ └── iife-modules/
└── web-platform/ # Concepts 9-10
├── dom/
└── http-fetch/
If a concept doesn't have tests:
| Resource | URL | Use For |
|---|---|---|
| MDN Web Docs | https://developer.mozilla.org | API docs, guides, compatibility |
| ECMAScript Spec | https://tc39.es/ecma262 | Authoritative behavior |
| TC39 Proposals | https://github.com/tc39/proposals | New features, stages |
| Can I Use | https://caniuse.com | Browser compatibility |
| Node.js Docs | https://nodejs.org/docs | Node-specific APIs |
| V8 Blog | https://v8.dev/blog | Engine internals |
| Resource | Path | Use For |
|---|---|---|
| Test Suite | /tests/ | Verify code examples |
| Concept Pages | /docs/concepts/ | Current content |
| Run Tests | npm test | Execute all tests |
Use this template to document your findings.
# Fact Check Report: [Concept Name]
**File:** `/docs/concepts/[slug].mdx`
**Date:** YYYY-MM-DD
**Reviewer:** [Name/Claude]
**Overall Status:** ✅ Verified | ⚠️ Minor Issues | ❌ Major Issues
---
## Executive Summary
[2-3 sentence summary of findings. State whether the page is accurate overall and highlight any critical issues.]
**Tests Run:** Yes/No
**Test Results:** X passing, Y failing
**External Links Checked:** X/Y valid
---
## Phase 1: Code Example Verification
| # | Description | Line | Status | Notes |
|---|-------------|------|--------|-------|
| 1 | [Brief description] | XX | ✅/⚠️/❌ | [Notes] |
| 2 | [Brief description] | XX | ✅/⚠️/❌ | [Notes] |
| 3 | [Brief description] | XX | ✅/⚠️/❌ | [Notes] |
### Code Issues Found
#### Issue 1: [Title]
**Location:** Line XX
**Severity:** Critical/Major/Minor
**Current Code:**
```javascript
// The problematic code
Problem: [Explanation of what's wrong] Correct Code:
// The corrected code
| Claim | Location | Source | Status | Notes |
|---|---|---|---|---|
| [Claim made] | Line XX | MDN/Spec | ✅/⚠️/❌ | [Notes] |
| Link Text | URL | Status |
|---|---|---|
| [Text] | [URL] | ✅ 200 / ❌ 404 |
[If any claims don't match the ECMAScript spec, detail them here]
| Resource | Type | Link | Content | Notes |
|---|---|---|---|---|
| [Title] | Article/Video | ✅/❌ | ✅/⚠️/❌ | [Notes] |
| Resource | Description Accurate? | Notes |
|---|---|---|
| [Title] | ✅/❌ | [Notes] |
| Claim | Location | Verdict | Notes |
|---|---|---|---|
| "[Claim]" | Line XX | ✅/⚠️/❌ | [Notes] |
Test File: /tests/[category]/[concept]/[concept].test.js
Tests Run: XX
Passing: XX
Failing: XX
| Test Name | Expected | Actual | Related Doc Line |
|---|---|---|---|
| [Test] | [Expected] | [Actual] | Line XX |
Examples in documentation without corresponding tests:
Verified by: [Name/Claude] Date: YYYY-MM-DD Recommendation: ✅ Ready to publish | ⚠️ Fix issues first | ❌ Major revision needed
---
## Quick Reference: Verification Commands
```bash
# Run all tests
npm test
# Run specific concept tests
npm test -- tests/fundamentals/call-stack/
# Check for broken links (if you have a link checker)
# Install: npm install -g broken-link-checker
# Run: blc https://developer.mozilla.org/... -ro
# Quick JavaScript REPL for testing
node
> typeof null
'object'
> [1,2,3].map(x => x * 2)
[ 2, 4, 6 ]
When fact-checking a concept page:
npm test catches code errors automaticallyRemember: Our readers trust us to teach them correct JavaScript. A single piece of misinformation can create confusion that takes years to unlearn. Take fact-checking seriously.
Create or update AgentSkills. Use when designing, structuring, or packaging skills with scripts, references, and assets.
Create or update AgentSkills. Use when designing, structuring, or packaging skills with scripts, references, and assets.
Set up and use 1Password CLI (op). Use when installing the CLI, enabling desktop app integration, signing in (single or multi-account), or reading/injecting/running secrets via op.
CLI to manage emails via IMAP/SMTP. Use `himalaya` to list, read, write, reply, forward, search, and organize emails from the terminal. Supports multiple accounts and message composition with MML (MIME Meta Language).
Create or update AgentSkills. Use when designing, structuring, or packaging skills with scripts, references, and assets.
Create or update AgentSkills. Use when designing, structuring, or packaging skills with scripts, references, and assets.
Set up and use 1Password CLI (op). Use when installing the CLI, enabling desktop app integration, signing in (single or multi-account), or reading/injecting/running secrets via op.
CLI to manage emails via IMAP/SMTP. Use `himalaya` to list, read, write, reply, forward, search, and organize emails from the terminal. Supports multiple accounts and message composition with MML (MIME Meta Language).