🐛 rippling: populate Internal struct when resolving resources by id

initRippling{Employee,Department,Team,Group} populated scalar args but
left the Internal struct empty, so a resource resolved directly by id
(e.g. rippling.employee(id: "x").manager) lost every cross-reference —
manager / department / team / workLocation / parent / parentTeam resolved
to null and group members to an empty list.

Each init now builds the resource via the newMqlRippling* constructor,
which sets the Internal struct, and returns it directly.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: tas50

providers/rippling/resources/department.go

@@ -42,9 +42,13 @@ func initRipplingDepartment(runtime *plugin.Runtime, args map[string]*llx.RawDat
 		if departments[i].ID != id {
 			continue
 		}
-		args["id"] = llx.StringData(departments[i].ID)
-		args["name"] = llx.StringData(departments[i].Name)
-		return args, nil, nil
+		// Build the resource directly so the Internal struct's parentID is
+		// populated — otherwise parent() would resolve to null.
+		dept, err := newMqlRipplingDepartment(runtime, &departments[i])
+		if err != nil {
+			return nil, nil, err
+		}
+		return args, dept, nil
 	}
 	return nil, nil, errors.New("rippling.department with id " + id + " not accessible with the configured token")
 }

providers/rippling/resources/employee.go

@@ -46,8 +46,14 @@ func initRipplingEmployee(runtime *plugin.Runtime, args map[string]*llx.RawData)
 		if employees[i].ID != id {
 			continue
 		}
-		populateEmployeeArgs(args, &employees[i])
-		return args, nil, nil
+		// Build the resource directly so the Internal struct's manager /
+		// department / team / work-location IDs are populated — otherwise
+		// those cross-references would resolve to null.
+		emp, err := newMqlRipplingEmployee(runtime, &employees[i])
+		if err != nil {
+			return nil, nil, err
+		}
+		return args, emp, nil
 	}
 	return nil, nil, errors.New("rippling.employee with id " + id + " not accessible with the configured token")
 }

providers/rippling/resources/group.go

@@ -42,10 +42,13 @@ func initRipplingGroup(runtime *plugin.Runtime, args map[string]*llx.RawData) (m
 		if groups[i].ID != id {
 			continue
 		}
-		args["id"] = llx.StringData(groups[i].ID)
-		args["name"] = llx.StringData(groups[i].Name)
-		args["version"] = llx.StringData(groups[i].Version)
-		return args, nil, nil
+		// Build the resource directly so the Internal struct's memberIDs are
+		// populated — otherwise members() would resolve to an empty list.
+		g, err := newMqlRipplingGroup(runtime, &groups[i])
+		if err != nil {
+			return nil, nil, err
+		}
+		return args, g, nil
 	}
 	return nil, nil, errors.New("rippling.group with id " + id + " not accessible with the configured token")
 }

providers/rippling/resources/team.go

@@ -42,9 +42,13 @@ func initRipplingTeam(runtime *plugin.Runtime, args map[string]*llx.RawData) (ma
 		if teams[i].ID != id {
 			continue
 		}
-		args["id"] = llx.StringData(teams[i].ID)
-		args["name"] = llx.StringData(teams[i].Name)
-		return args, nil, nil
+		// Build the resource directly so the Internal struct's parentTeamID
+		// is populated — otherwise parentTeam() would resolve to null.
+		t, err := newMqlRipplingTeam(runtime, &teams[i])
+		if err != nil {
+			return nil, nil, err
+		}
+		return args, t, nil
 	}
 	return nil, nil, errors.New("rippling.team with id " + id + " not accessible with the configured token")
 }