fix: remove HS grade support — use grades 9-12 directly

API confirmed HS standards have gradeLevel=['9','10','11','12'];
the KG API does not accept gradeLevel=HS as a query parameter.

- Remove 'HS' from SUPPORTED_GRADES, VALID_GRADES, input_schema.json
- Remove HS prefix parsing from parseGradeFromStandard
  (HSA-APR.A.1 now throws ValidationError — callers should pass grade='9')
- Remove dead client-side MP normalizedStatementType filter
  (API never returns Mathematical Practice for Mathematics subject queries)
- Simplify README to bare minimum

Author: adnanrhussain

evals/prompts/math/standards-alignment/input_schema.json

@@ -14,11 +14,11 @@
     "grade": {
       "type": "string",
       "description": "Grade level the question targets.",
-      "enum": ["K", "1", "2", "3", "4", "5", "6", "7", "8", "9", "10", "11", "12", "HS"]
+      "enum": ["K", "1", "2", "3", "4", "5", "6", "7", "8", "9", "10", "11", "12"]
     },
     "statementCode": {
       "type": "string",
-      "description": "CCSS math standard code (e.g. '3.MD.C.7.d', 'HSA-APR.A.1').",
+      "description": "CCSS math standard code (e.g. '3.MD.C.7.d', 'K.CC.A.1').",
       "minLength": 1,
       "maxLength": 50
     }

sdks/typescript/src/knowledge-graph/README.md

@@ -1,22 +1,12 @@
 # Knowledge Graph client
 
-HTTP client for the [Learning Commons Knowledge Graph API](https://api.learningcommons.org/knowledge-graph/v0), used by `MathStandardsAlignmentEvaluator` to fetch academic standards and learning components.
+HTTP client for the [Learning Commons Knowledge Graph API](https://api.learningcommons.org/knowledge-graph/v0).
 
-## Files
-
-| File | Purpose |
-|---|---|
-| `kg-api.d.ts` | **Generated.** TypeScript types from the KG OpenAPI spec — do not edit by hand |
-| `types.ts` | Internal types: `AcademicStandard`, `LearningComponent`, `StandardInfo`, `parseGradeFromStandard` |
-| `client.ts` | `KnowledgeGraphClient` — HTTP calls, concurrency limiting, promise caching |
-| `index.ts` | Barrel |
-
-## Regenerating `kg-api.d.ts`
+## Regenerate types
 
 ```bash
 npm run generate:kg-types
 ```
 
-Pulls the spec from `https://docs.learningcommons.org/api-reference/knowledge-graph-api/openapi.yaml`.
-
-> **Note on spec enums:** `normalizedStatementType` and `gradeLevel` use `string` in our interfaces rather than the spec's enum types — the spec enums are incomplete (omit `"Mathematical Practice"` and `"HS"`).
+Pulls from `https://docs.learningcommons.org/api-reference/knowledge-graph-api/openapi.yaml`.
+`kg-api.d.ts` is generated — do not edit by hand.

sdks/typescript/src/knowledge-graph/client.ts

@@ -99,11 +99,19 @@ export class KnowledgeGraphClient {
       `&gradeLevel=${encodeURIComponent(grade)}` +
       `&normalizedStatementType=Standard`;
 
-    const data = (await kgFetch(url, this.apiKey)) as { data: Array<AcademicStandard> };
+    const data = (await kgFetch(url, this.apiKey)) as {
+      data: Array<AcademicStandard>;
+      pagination?: { hasMore: boolean; nextCursor: string | null };
+    };
 
-    return (data.data ?? [])
-      .filter((item) => item.normalizedStatementType !== 'Mathematical Practice')
-      .map((item) => ({
+    if (data.pagination?.hasMore) {
+      throw new KnowledgeGraphError(
+        `getStandardsByGrade returned a paginated result for grade "${grade}" — ` +
+        `increase limit or implement cursor pagination to retrieve all standards.`,
+      );
+    }
+
+    return (data.data ?? []).map((item) => ({
         caseIdentifierUUID: item.caseIdentifierUUID,
         statementCode: item.statementCode,
         description: item.description,

sdks/typescript/src/knowledge-graph/types.ts

@@ -1,8 +1,9 @@
 import { ValidationError } from '../errors.js';
 
-// The generated spec types are in kg-api.d.ts — see npm run generate:kg-types.
-// We use string for normalizedStatementType and gradeLevel because the spec's
-// enum types are incomplete (omit "Mathematical Practice" and "HS").
+// Generated spec types are in kg-api.d.ts — see npm run generate:kg-types.
+// We use string for normalizedStatementType and gradeLevel rather than the
+// spec's strict enum types so the interfaces remain compatible if the API
+// returns values outside the current spec.
 
 /**
  * An academic standard from the LC Knowledge Graph (spec: StandardsFrameworkItem).
@@ -33,12 +34,9 @@ export interface StandardInfo {
 
 // ---------------------------------------------------------------------------
 
-const VALID_GRADES = new Set(['K', '1', '2', '3', '4', '5', '6', '7', '8', '9', '10', '11', '12', 'HS']);
+const VALID_GRADES = new Set(['K', '1', '2', '3', '4', '5', '6', '7', '8', '9', '10', '11', '12']);
 
 export function parseGradeFromStandard(code: string): string {
-  // High school standards use a domain letter + hyphen: HSA-APR.A.1, HSF-BF.A.1, etc.
-  if (/^HS[A-Z]-/.test(code)) return 'HS';
-
   const match = code.match(/^(K|\d+)\./);
   if (!match || !VALID_GRADES.has(match[1])) {
     throw new ValidationError(`Cannot parse grade from standard code: "${code}"`);

sdks/typescript/tests/unit/evaluators/math/standards-alignment.test.ts

@@ -170,20 +170,6 @@ describe('MathStandardsAlignmentEvaluator - evaluate', () => {
     await expect(evaluator.evaluate(QUESTION, GRADE, '')).rejects.toThrow(ValidationError);
   });
 
-  it('accepts grade HS with a high school standard code', async () => {
-    const hsRepo = makeMockKgClient({
-      getLearningComponentsByCode: vi.fn().mockResolvedValue({ uuid: 'uuid-abc', components: [{ description: 'Factor polynomials' }] }),
-    });
-    vi.mocked(mockProvider.generateStructured).mockResolvedValue({
-      ...MOCK_BATCH_RESPONSE,
-      data: { evaluations: [{ reasoning: 'ok', aligned: true, feedback: '' }] },
-    });
-    const evaluator = new MathStandardsAlignmentEvaluator(makeConfig({ _kgClient: hsRepo }));
-    const result = await evaluator.evaluate('Factor x² + 5x + 6.', 'HS', 'HSA-APR.A.1');
-    expect(result.grade).toBe('HS');
-    expect(result.statementCode).toBe('HSA-APR.A.1');
-  });
-
   it('correctly handles grade K standard', async () => {
     const kRepo = makeMockKgClient({
       getLearningComponentsByCode: vi.fn().mockResolvedValue({ uuid: 'uuid-abc', components: [{ description: 'Count objects' }] }),
@@ -516,12 +502,6 @@ describe('parseGradeFromStandard', () => {
     ['3.MD.C.7.d', '3'],
     ['10.NBT.A.1', '10'],
     ['12.F.BF.1', '12'],
-    // High school standards use domain letter + hyphen, not a numeric grade prefix
-    ['HSA-APR.A.1', 'HS'],
-    ['HSF-BF.A.1', 'HS'],
-    ['HSG-CO.A.1', 'HS'],
-    ['HSN-CN.A.1', 'HS'],
-    ['HSS-ID.A.1', 'HS'],
   ])('%s → %s', (code, expected) => {
     expect(parseGradeFromStandard(code)).toBe(expected);
   });
@@ -531,7 +511,7 @@ describe('parseGradeFromStandard', () => {
     ['ZZ.OA.A.1'],
     ['no-dots'],
     ['13.MD.C.7'],
-    ['HSA.APR.A.1'], // missing hyphen — not a valid HS code
+    ['HSA-APR.A.1'], // HS codes not supported — use grade 9/10/11/12 directly
   ])('throws ValidationError for "%s"', (code) => {
     expect(() => parseGradeFromStandard(code)).toThrow(ValidationError);
   });

sdks/typescript/tests/unit/knowledge-graph/client.test.ts

@@ -134,31 +134,21 @@ describe('KnowledgeGraphClient - getLearningComponents', () => {
 });
 
 describe('KnowledgeGraphClient - getStandardsByGrade', () => {
-  it('excludes Mathematical Practice standards', async () => {
-    mockFetch(200, { data: [
+  it('returns standards filtered to normalizedStatementType=Standard via URL', async () => {
+    const fetchMock = vi.fn().mockResolvedValue({ ok: true, status: 200, json: () => Promise.resolve({ data: [
       { caseIdentifierUUID: 'u1', statementCode: '3.MD.C.7.d', normalizedStatementType: 'Standard', gradeLevel: ['3'] },
-      { caseIdentifierUUID: 'u2', statementCode: '3.MP.1', normalizedStatementType: 'Mathematical Practice', gradeLevel: ['3'] },
-    ]});
-    const results = await new KnowledgeGraphClient(API_KEY).getStandardsByGrade('3');
-    expect(results.map((s) => s.statementCode)).toContain('3.MD.C.7.d');
-    expect(results.map((s) => s.statementCode)).not.toContain('3.MP.1');
-  });
-
-  it('always includes normalizedStatementType=Standard in URL', async () => {
-    const fetchMock = vi.fn().mockResolvedValue({ ok: true, status: 200, json: () => Promise.resolve({ data: [] }), text: () => Promise.resolve('') });
+    ] }), text: () => Promise.resolve('') });
     vi.stubGlobal('fetch', fetchMock);
-    await new KnowledgeGraphClient(API_KEY).getStandardsByGrade('3');
+    const results = await new KnowledgeGraphClient(API_KEY).getStandardsByGrade('3');
     expect(fetchMock.mock.calls[0][0]).toContain('normalizedStatementType=Standard');
+    expect(results).toHaveLength(1);
+    expect(results[0].statementCode).toBe('3.MD.C.7.d');
   });
 
-  it('uses normalizedStatementType not statementCode string to exclude MP', async () => {
-    mockFetch(200, { data: [
-      { caseIdentifierUUID: 'u1', statementCode: 'HSS-IC.MP.2', normalizedStatementType: 'Standard', gradeLevel: ['HS'] },
-      { caseIdentifierUUID: 'u2', statementCode: '3.MP.1', normalizedStatementType: 'Mathematical Practice', gradeLevel: ['3'] },
-    ]});
-    const results = await new KnowledgeGraphClient(API_KEY).getStandardsByGrade('HS');
-    expect(results.map((s) => s.statementCode)).toContain('HSS-IC.MP.2');
-    expect(results.map((s) => s.statementCode)).not.toContain('3.MP.1');
+  it('throws KnowledgeGraphError if hasMore=true (pagination not implemented for standards)', async () => {
+    mockFetch(200, { data: [], pagination: { hasMore: true, nextCursor: 'page-2' } });
+    await expect(new KnowledgeGraphClient(API_KEY).getStandardsByGrade('3'))
+      .rejects.toThrow(KnowledgeGraphError);
   });
 });