Revert "Fix breaking APIs (#45)"

This reverts commit 2cc4322.

Author: Fodoj

docs/data-sources/organization_user.md

@@ -22,7 +22,7 @@ description: |-
 
 ### Read-Only
 
-- `created` (Number) The Unix timestamp when the user was added to the organization
+- `added_at` (Number) The Unix timestamp when the user was added to the organization
 - `id` (String) The ID of this resource.
 - `name` (String) The name of the user
 - `role` (String) The role of the user in the organization (owner, member, or reader)

docs/data-sources/organization_users.md

@@ -46,7 +46,7 @@ output "all_users_count" {
 
 Read-Only:
 
-- `created` (Number)
+- `added_at` (Number)
 - `email` (String)
 - `id` (String)
 - `name` (String)

docs/data-sources/project.md

@@ -28,7 +28,7 @@ output "project_id" {
 # Use project data to set variables
 locals {
   project_active = data.openai_project.production.status == "active"
-  project_title  = data.openai_project.production.title
+  project_name   = data.openai_project.production.name
 }
 ```
 
@@ -45,9 +45,17 @@ locals {
 
 ### Read-Only
 
-- `archived_at` (Number) Timestamp when the project was archived (null if not archived)
-- `created` (Number) Timestamp when the project was created
+- `created_at` (Number) Timestamp when the project was created
 - `id` (String) The ID of this resource.
-- `is_initial` (Boolean) Whether this is the initial project
+- `name` (String) The name of the project
 - `status` (String) The status of the project
-- `title` (String) The title of the project
+- `usage_limits` (List of Object) Usage limits for the project (see [below for nested schema](#nestedatt--usage_limits))
+
+<a id="nestedatt--usage_limits"></a>
+### Nested Schema for `usage_limits`
+
+Read-Only:
+
+- `max_monthly_dollars` (Number)
+- `max_parallel_requests` (Number)
+- `max_tokens` (Number)

docs/data-sources/projects.md

@@ -42,9 +42,7 @@ output "project_count" {
 
 Read-Only:
 
-- `archived_at` (Number)
-- `created` (Number)
+- `created_at` (Number)
 - `id` (String)
-- `is_initial` (Boolean)
+- `name` (String)
 - `status` (String)
-- `title` (String)

docs/resources/organization_user.md

@@ -41,7 +41,7 @@ resource "openai_organization_user" "organization_users" {
 
 ### Read-Only
 
-- `created` (Number) The Unix timestamp when the user was added to the organization
+- `added_at` (Number) The Unix timestamp when the user was added to the organization
 - `email` (String) The email address of the user
 - `id` (String) The ID of this resource.
 - `name` (String) The name of the user

docs/resources/project.md

@@ -15,12 +15,14 @@ description: |-
 ```terraform
 # Create a new OpenAI project
 resource "openai_project" "development" {
-  title = "Development Project"
+  name        = "Development Project"
+  description = "Project for development and testing purposes"
 }
 
 # Create a production project
 resource "openai_project" "production" {
-  title = "Production API Services"
+  name        = "Production API Services"
+  description = "Project for production API services and deployments"
 }
 
 # Output the project ID
@@ -35,18 +37,18 @@ output "dev_project_id" {
 
 ### Required
 
-- `title` (String) The title of the project
+- `name` (String) The name of the project
 
 ### Optional
 
+- `description` (String) A description of the project
 - `timeouts` (Block, Optional) (see [below for nested schema](#nestedblock--timeouts))
 
 ### Read-Only
 
-- `archived_at` (Number) Timestamp when the project was archived, if applicable
-- `created` (Number) Timestamp when the project was created
+- `archived_at` (String) Timestamp when the project was archived, if applicable
+- `created_at` (String) Timestamp when the project was created
 - `id` (String) The ID of this resource.
-- `is_initial` (Boolean) Whether this is the initial project
 - `status` (String) Status of the project (active, archived, etc.)
 
 <a id="nestedblock--timeouts"></a>

examples/data-sources/openai_project/data-source.tf

@@ -13,5 +13,5 @@ output "project_id" {
 # Use project data to set variables
 locals {
   project_active = data.openai_project.production.status == "active"
-  project_title  = data.openai_project.production.title
+  project_name   = data.openai_project.production.name
 }

examples/resources/openai_project/resource.tf

@@ -1,11 +1,13 @@
 # Create a new OpenAI project
 resource "openai_project" "development" {
-  title = "Development Project"
+  name        = "Development Project"
+  description = "Project for development and testing purposes"
 }
 
 # Create a production project
 resource "openai_project" "production" {
-  title = "Production API Services"
+  name        = "Production API Services"
+  description = "Project for production API services and deployments"
 }
 
 # Output the project ID

internal/client/client.go

@@ -141,13 +141,19 @@ func (c *OpenAIClient) SetTimeout(timeout time.Duration) {
 
 // Project represents a project in OpenAI
 type Project struct {
-	ID         string `json:"id"`
-	Object     string `json:"object"`
-	Created    int64  `json:"created"`
-	Status     string `json:"status"`
-	ArchivedAt *int64 `json:"archived_at"`
-	IsInitial  bool   `json:"is_initial"`
-	Title      string `json:"title"`
+	Object         string        `json:"object"`
+	ID             string        `json:"id"`
+	Name           string        `json:"name"`
+	Description    string        `json:"description,omitempty"`
+	OrganizationID string        `json:"organization_id,omitempty"`
+	CreatedAt      *int64        `json:"created_at"`
+	ArchivedAt     *int64        `json:"archived_at"`
+	Status         string        `json:"status"`
+	IsDefault      bool          `json:"is_default,omitempty"`
+	BillingMode    string        `json:"billing_mode,omitempty"`
+	APIKeys        []APIKey      `json:"api_keys,omitempty"`
+	RateLimits     []RateLimit   `json:"rate_limits,omitempty"`
+	Users          []ProjectUser `json:"users,omitempty"`
 }
 
 // ProjectUser represents a user associated with a project
@@ -191,7 +197,9 @@ type APIKey struct {
 
 // CreateProjectRequest represents the request to create a project
 type CreateProjectRequest struct {
-	Title string `json:"title"`
+	Name        string `json:"name"`
+	Description string `json:"description,omitempty"`
+	IsDefault   bool   `json:"is_default,omitempty"`
 }
 
 // Error represents an error from the OpenAI API
@@ -559,64 +567,23 @@ type AssistantFunction struct {
 	Parameters  json.RawMessage `json:"parameters"`
 }
 
-// UserInfo represents the nested user details in an organization user response
-type UserInfo struct {
-	ID     string `json:"id"`
-	Object string `json:"object"`
-	Email  string `json:"email"`
-	Name   string `json:"name"`
-}
-
-// OrganizationUser represents an OpenAI organization user as returned by the API
-type OrganizationUser struct {
-	Object                         string   `json:"object"`
-	Created                        int64    `json:"created"`
-	IsDefault                      bool     `json:"is_default"`
-	IsScaleTierAuthorizedPurchaser bool     `json:"is_scale_tier_authorized_purchaser"`
-	IsScimManaged                  bool     `json:"is_scim_managed"`
-	IsServiceAccount               bool     `json:"is_service_account"`
-	Role                           string   `json:"role"`
-	User                           UserInfo `json:"user"`
-}
-
-// User provides a flattened view of OrganizationUser for easier access
-// This is a convenience wrapper used internally
+// User represents an OpenAI user
 type User struct {
-	ID                             string `json:"id"`
-	Object                         string `json:"object"`
-	Email                          string `json:"email"`
-	Name                           string `json:"name"`
-	Role                           string `json:"role"`
-	Created                        int64  `json:"created"`
-	IsDefault                      bool   `json:"is_default"`
-	IsScaleTierAuthorizedPurchaser bool   `json:"is_scale_tier_authorized_purchaser"`
-	IsScimManaged                  bool   `json:"is_scim_managed"`
-	IsServiceAccount               bool   `json:"is_service_account"`
-}
-
-// ToUser converts an OrganizationUser to a flattened User struct
-func (ou *OrganizationUser) ToUser() *User {
-	return &User{
-		ID:                             ou.User.ID,
-		Object:                         ou.Object,
-		Email:                          ou.User.Email,
-		Name:                           ou.User.Name,
-		Role:                           ou.Role,
-		Created:                        ou.Created,
-		IsDefault:                      ou.IsDefault,
-		IsScaleTierAuthorizedPurchaser: ou.IsScaleTierAuthorizedPurchaser,
-		IsScimManaged:                  ou.IsScimManaged,
-		IsServiceAccount:               ou.IsServiceAccount,
-	}
+	ID      string `json:"id"`
+	Object  string `json:"object"`
+	Email   string `json:"email"`
+	Name    string `json:"name"`
+	Role    string `json:"role"`
+	AddedAt int64  `json:"added_at"`
 }
 
 // UsersResponse represents the response from the list users API
 type UsersResponse struct {
-	Object  string             `json:"object"`
-	Data    []OrganizationUser `json:"data"`
-	FirstID string             `json:"first_id"`
-	LastID  string             `json:"last_id"`
-	HasMore bool               `json:"has_more"`
+	Object  string `json:"object"`
+	Data    []User `json:"data"`
+	FirstID string `json:"first_id"`
+	LastID  string `json:"last_id"`
+	HasMore bool   `json:"has_more"`
 }
 
 // ListUsers retrieves a list of users in the organization
@@ -693,13 +660,12 @@ func (c *OpenAIClient) FindUserByEmail(email string) (*User, bool, error) {
 		return nil, false, nil
 	}
 
-	// Return the first matching user (convert from OrganizationUser to User)
-	orgUser := usersResponse.Data[0]
-	user := orgUser.ToUser()
+	// Return the first matching user
+	user := usersResponse.Data[0]
 
 	// Double check that the email matches exactly (case insensitive)
 	if strings.EqualFold(user.Email, email) {
-		return user, true, nil
+		return &user, true, nil
 	}
 
 	// No exact match found
@@ -725,14 +691,13 @@ func (c *OpenAIClient) GetUser(userID string) (*User, bool, error) {
 		return nil, false, fmt.Errorf("error getting user: %w", err)
 	}
 
-	// Parse the response into OrganizationUser (new API format)
-	var orgUser OrganizationUser
-	if err := json.Unmarshal(respBody, &orgUser); err != nil {
+	// Parse the response
+	var user User
+	if err := json.Unmarshal(respBody, &user); err != nil {
 		return nil, false, fmt.Errorf("error decoding user response: %w", err)
 	}
 
-	// Convert to flattened User struct
-	return orgUser.ToUser(), true, nil
+	return &user, true, nil
 }
 
 // UpdateUserRole updates a user's role
@@ -754,14 +719,13 @@ func (c *OpenAIClient) UpdateUserRole(userID string, role string) (*User, error)
 		return nil, fmt.Errorf("error updating user role: %w", err)
 	}
 
-	// Parse the response into OrganizationUser (new API format)
-	var orgUser OrganizationUser
-	if err := json.Unmarshal(respBody, &orgUser); err != nil {
+	// Parse the response
+	var user User
+	if err := json.Unmarshal(respBody, &user); err != nil {
 		return nil, fmt.Errorf("error decoding user response: %w", err)
 	}
 
-	// Convert to flattened User struct
-	return orgUser.ToUser(), nil
+	return &user, nil
 }
 
 // DeleteUser removes a user from the organization
@@ -1111,15 +1075,25 @@ func (c *OpenAIClient) ListProjects(limit int, includeArchived bool, after strin
 	return &resp, nil
 }
 
-// CreateProject creates a new project with the given title
-func (c *OpenAIClient) CreateProject(title string) (*Project, error) {
+// CreateProject creates a new project with the given name and description
+func (c *OpenAIClient) CreateProject(name, description string, isDefault bool) (*Project, error) {
 	// Create the request body
 	requestBody := map[string]interface{}{
-		"title": title,
+		"name": name,
+	}
+
+	// Only include description if it's not empty
+	if description != "" {
+		requestBody["description"] = description
+	}
+
+	// Only include is_default if it's true (don't send false)
+	if isDefault {
+		requestBody["is_default"] = isDefault
 	}
 
 	// Debug information
-	fmt.Printf("Creating project with title: %s\n", title)
+	fmt.Printf("Creating project with name: %s\n", name)
 	fmt.Printf("Request body: %+v\n", requestBody)
 
 	// Use the exact endpoint from the curl command that works
@@ -1166,11 +1140,21 @@ func (c *OpenAIClient) GetProject(id string) (*Project, error) {
 	return &project, nil
 }
 
-// UpdateProject updates an existing project with the given title
-func (c *OpenAIClient) UpdateProject(id, title string) (*Project, error) {
+// UpdateProject updates an existing project with the given details
+func (c *OpenAIClient) UpdateProject(id, name, description string, isDefault bool) (*Project, error) {
 	// Create the request body
 	requestBody := map[string]interface{}{
-		"title": title,
+		"name": name,
+	}
+
+	// Only include description if it's not empty
+	if description != "" {
+		requestBody["description"] = description
+	}
+
+	// Only include is_default if it's true (don't send false)
+	if isDefault {
+		requestBody["is_default"] = isDefault
 	}
 
 	// Use the exact endpoint structure consistent with the curl command

internal/provider/data_source_openai_organization_user.go

@@ -38,7 +38,7 @@ func dataSourceOpenAIOrganizationUser() *schema.Resource {
 				Computed:    true,
 				Description: "The role of the user in the organization (owner, member, or reader)",
 			},
-			"created": {
+			"added_at": {
 				Type:        schema.TypeInt,
 				Computed:    true,
 				Description: "The Unix timestamp when the user was added to the organization",
@@ -114,8 +114,8 @@ func dataSourceOpenAIOrganizationUserRead(ctx context.Context, d *schema.Resourc
 		return diag.FromErr(fmt.Errorf("error setting role: %s", err))
 	}
 
-	if err := d.Set("created", user.Created); err != nil {
-		return diag.FromErr(fmt.Errorf("error setting created: %s", err))
+	if err := d.Set("added_at", user.AddedAt); err != nil {
+		return diag.FromErr(fmt.Errorf("error setting added_at: %s", err))
 	}
 
 	return nil