@forwardimpact/schema 0.2.0 → 0.3.0

package/examples/disciplines/data_engineering.yaml

@@ -59,18 +59,6 @@ agent:
   priority: |
     Data quality is paramount. Always validate data at ingestion points and
     document schema assumptions. Treat undocumented schemas with suspicion.
-  beforeMakingChanges:
-    - Understand the data sources and their quality characteristics
-    - Map data lineage and dependencies
-    - Consider performance implications for downstream consumers
-  delegation: |
-    When facing tasks outside your expertise, use `runSubagent` to delegate:
-    - Statistical analysis or model validation → data science subagent
-    - Application code changes → software engineering subagent
-    - Infrastructure or deployment → SRE subagent
-
-    Subagents run in isolated context. Provide clear task descriptions and
-    specify what information to return.
   constraints:
     - Ignoring data quality issues
     - Creating pipelines without proper error handling

package/examples/disciplines/engineering_management.yaml

@@ -53,10 +53,6 @@ agent:
     remove organizational friction.
 
     Your role is to support engineering judgment, not replace it.
-  beforeMakingChanges:
-    - Consider team dynamics and individual growth
-    - Think about systemic improvements over individual fixes
-    - Focus on enabling others rather than doing directly
   constraints:
     - Focus on enabling and empowering rather than doing directly
     - Consider long-term team health over short-term fixes

package/examples/disciplines/software_engineering.yaml

@@ -59,18 +59,6 @@ agent:
     Code review is more important than code generation. Every line of code
     you produce must be understood and verified. Never ship code without
     comprehensive testing.
-  beforeMakingChanges:
-    - Understand the existing architecture and patterns
-    - Identify test coverage requirements
-    - Consider backward compatibility implications
-  delegation: |
-    When facing tasks outside your expertise, use `runSubagent` to delegate:
-    - Data modeling or statistical analysis → data science subagent
-    - Security assessment or threat modeling → research subagent
-    - Complex debugging across unfamiliar systems → research subagent
-
-    Subagents run in isolated context. Provide clear task descriptions and
-    specify what information to return.
   constraints:
     - Committing code without running tests
     - Making changes without understanding the existing codebase

package/examples/tracks/platform.yaml

@@ -37,11 +37,6 @@ agent:
 
     Every API change must consider developer experience. Treat breaking 
     changes with extreme caution—your consumers build on your stability.
-  beforeMakingChanges:
-    - Understand the existing architecture and patterns
-    - Identify test coverage requirements
-    - Consider backward compatibility implications
-    - Plan documentation updates
   constraints:
     - Maintain backward compatibility
     - Document breaking changes with migration guides

package/examples/tracks/sre.yaml

@@ -36,11 +36,6 @@ agent:
 
     Production stability trumps feature velocity. No change ships without
     understanding its impact on system reliability.
-  beforeMakingChanges:
-    - Understand system dependencies and failure modes
-    - Ensure observability instrumentation is in place
-    - Consider rollback procedures and blast radius
-    - Verify runbooks are updated
   constraints:
     - Never compromise on observability instrumentation
     - Document all runbooks and incident response procedures

package/lib/validation.js

@@ -750,48 +750,6 @@ function validateDiscipline(
       );
     }
 
-    // Optional: beforeMakingChanges (array of strings)
-    if (discipline.agent.beforeMakingChanges !== undefined) {
-      if (!Array.isArray(discipline.agent.beforeMakingChanges)) {
-        errors.push(
-          createError(
-            "INVALID_VALUE",
-            "Discipline agent beforeMakingChanges must be an array",
-            `${agentPath}.beforeMakingChanges`,
-            discipline.agent.beforeMakingChanges,
-          ),
-        );
-      } else {
-        discipline.agent.beforeMakingChanges.forEach((item, i) => {
-          if (typeof item !== "string") {
-            errors.push(
-              createError(
-                "INVALID_VALUE",
-                "Discipline agent beforeMakingChanges items must be strings",
-                `${agentPath}.beforeMakingChanges[${i}]`,
-                item,
-              ),
-            );
-          }
-        });
-      }
-    }
-
-    // Optional: delegation (string)
-    if (
-      discipline.agent.delegation !== undefined &&
-      typeof discipline.agent.delegation !== "string"
-    ) {
-      errors.push(
-        createError(
-          "INVALID_VALUE",
-          "Discipline agent delegation must be a string",
-          `${agentPath}.delegation`,
-          discipline.agent.delegation,
-        ),
-      );
-    }
-
     // Optional: constraints (array of strings)
     if (discipline.agent.constraints !== undefined) {
       if (!Array.isArray(discipline.agent.constraints)) {
@@ -824,7 +782,7 @@ function validateDiscipline(
       errors.push(
         createError(
           "INVALID_FIELD",
-          "Discipline agent 'coreInstructions' field is not supported. Use identity, priority, beforeMakingChanges, and delegation instead.",
+          "Discipline agent 'coreInstructions' field is not supported. Use identity, priority, and constraints instead.",
           `${agentPath}.coreInstructions`,
         ),
       );
@@ -1024,33 +982,6 @@ function validateTrack(
       );
     }
 
-    // Optional: beforeMakingChanges (array of strings)
-    if (track.agent.beforeMakingChanges !== undefined) {
-      if (!Array.isArray(track.agent.beforeMakingChanges)) {
-        errors.push(
-          createError(
-            "INVALID_VALUE",
-            "Track agent beforeMakingChanges must be an array",
-            `${agentPath}.beforeMakingChanges`,
-            track.agent.beforeMakingChanges,
-          ),
-        );
-      } else {
-        track.agent.beforeMakingChanges.forEach((item, i) => {
-          if (typeof item !== "string") {
-            errors.push(
-              createError(
-                "INVALID_VALUE",
-                "Track agent beforeMakingChanges items must be strings",
-                `${agentPath}.beforeMakingChanges[${i}]`,
-                item,
-              ),
-            );
-          }
-        });
-      }
-    }
-
     // Optional: constraints (array of strings)
     if (track.agent.constraints !== undefined) {
       if (!Array.isArray(track.agent.constraints)) {
@@ -1083,7 +1014,7 @@ function validateTrack(
       errors.push(
         createError(
           "INVALID_FIELD",
-          "Track agent 'coreInstructions' field is not supported. Use identity, priority, beforeMakingChanges, and constraints instead.",
+          "Track agent 'coreInstructions' field is not supported. Use identity, priority, and constraints instead.",
           `${agentPath}.coreInstructions`,
         ),
       );

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forwardimpact/schema",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "Schema definitions and data loading for skills, behaviours, and role frameworks",
   "license": "Apache-2.0",
   "repository": {

package/schema/json/discipline.schema.json

@@ -111,15 +111,6 @@
           "type": "string",
           "description": "Priority guidance for the agent"
         },
-        "beforeMakingChanges": {
-          "type": "array",
-          "description": "Checklist items to consider before making changes",
-          "items": { "type": "string" }
-        },
-        "delegation": {
-          "type": "string",
-          "description": "Guidance on when and how to delegate to subagents"
-        },
         "constraints": {
           "type": "array",
           "description": "Things the agent should avoid or never do",

package/schema/json/track.schema.json

@@ -83,11 +83,6 @@
           "type": "string",
           "description": "Priority guidance specific to this track"
         },
-        "beforeMakingChanges": {
-          "type": "array",
-          "description": "Additional checklist items to consider",
-          "items": { "type": "string" }
-        },
         "constraints": {
           "type": "array",
           "description": "Additional constraints specific to this track",

package/schema/rdf/discipline.ttl

@@ -109,16 +109,6 @@ fit:priority a rdf:Property ;
     rdfs:comment "Priority guidance for the agent"@en ;
     rdfs:range xsd:string .
 
-fit:beforeMakingChanges a rdf:Property ;
-    rdfs:label "beforeMakingChanges"@en ;
-    rdfs:comment "Checklist items to consider before making changes"@en ;
-    rdfs:range xsd:string .
-
-fit:delegation a rdf:Property ;
-    rdfs:label "delegation"@en ;
-    rdfs:comment "Guidance on when and how to delegate to subagents"@en ;
-    rdfs:range xsd:string .
-
 # Modifier Properties
 fit:targetBehaviour a rdf:Property ;
     rdfs:label "targetBehaviour"@en ;
@@ -289,19 +279,6 @@ fit:DisciplineAgentSectionShape a sh:NodeShape ;
         sh:name "priority" ;
         sh:description "Priority guidance for the agent" ;
     ] ;
-    sh:property [
-        sh:path fit:beforeMakingChanges ;
-        sh:datatype xsd:string ;
-        sh:name "beforeMakingChanges" ;
-        sh:description "Checklist items to consider before making changes" ;
-    ] ;
-    sh:property [
-        sh:path fit:delegation ;
-        sh:datatype xsd:string ;
-        sh:maxCount 1 ;
-        sh:name "delegation" ;
-        sh:description "Guidance on when and how to delegate to subagents" ;
-    ] ;
     sh:property [
         sh:path fit:constraints ;
         sh:datatype xsd:string ;

package/schema/rdf/track.ttl

@@ -166,12 +166,6 @@ fit:TrackAgentSectionShape a sh:NodeShape ;
         sh:name "priority" ;
         sh:description "Priority guidance specific to this track" ;
     ] ;
-    sh:property [
-        sh:path fit:beforeMakingChanges ;
-        sh:datatype xsd:string ;
-        sh:name "beforeMakingChanges" ;
-        sh:description "Additional checklist items to consider" ;
-    ] ;
     sh:property [
         sh:path fit:constraints ;
         sh:datatype xsd:string ;