feat: add support for .report root field in CRD plugin (#322)

[misraved]

Example query results

Results

Add example SQL query results here (please include the input queries as well)

[Copilot]

Pull request overview

This PR adds support for the .report root field in custom resource definitions (CRDs), following the same pattern as the existing .spec and .status fields. This enables Kubernetes CRD plugins to extract and expose report-level properties as columns in Steampipe tables.

Key Changes:

• Added support for extracting and displaying report fields from custom resources
• Implemented column generation logic for report properties with prefix handling
• Added new extractReportProperty function to retrieve report field values

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Missing space after comma in function parameter list. Should be versionSchemaStatus interface{}, versionSchemaReport interface{} instead of versionSchemaStatus interface{},versionSchemaReport interface{}.

Suggested change

	func getCustomResourcesDynamicColumns(ctx context.Context, versionSchemaSpec interface{}, versionSchemaStatus interface{},versionSchemaReport interface{}) []*plugin.Column {
	func getCustomResourcesDynamicColumns(ctx context.Context, versionSchemaSpec interface{}, versionSchemaStatus interface{}, versionSchemaReport interface{}) []*plugin.Column {

[Copilot]

Potential panic due to type assertion without nil check. If versionSchemaReport is nil, this type assertion will panic. Consider adding a nil check similar to the existing pattern or handling the case where report schema might not be present in the CRD:

if versionSchemaReport != nil {
    schemaReport := versionSchemaReport.(v1.JSONSchemaProps)
    // ... rest of the report columns logic
}

[Copilot]

The report columns logic does not update the allColumns slice when a new column is added (flag == 0), unlike the spec columns logic at line 75. This could lead to duplicate column names if subsequent sections (like report) have properties that overlap with earlier sections. Consider adding:

if flag == 0 {
    column := &plugin.Column{
        Name:        strcase.ToSnake(k),
        Description: v.Description,
        Transform:   transform.FromP(extractReportProperty, k),
    }
    allColumns = append(allColumns, strcase.ToSnake(k))  // Add this line
    setDynamicColumns(v, column)
    columns = append(columns, column)
}

Suggested change

	}
	}
	allColumns = append(allColumns, strcase.ToSnake(k))