fix: adds created timestamp for openai text completions

[sammaji]

Summary

Adds the missing created field to BifrostTextCompletionResponse to align with the OpenAI text completion response schema, and modernizes the ExtraParams type alias from map[string]interface{} to the equivalent map[string]any.

Changes

• Added Created int field with omitempty to BifrostTextCompletionResponse, representing the Unix timestamp (in seconds) of when the completion was created — this field was previously absent from the struct despite being part of the API response.
• Replaced map[string]interface{} with map[string]any in TextCompletionParameters.ExtraParams to use the modern Go type alias.

Type of change

• Bug fix
• Feature
• Refactor
• Documentation
• Chore/CI

Affected areas

• Core (Go)
• Transports (HTTP)
• Providers/Integrations
• Plugins
• UI (React)
• Docs

How to test

go version
go test ./...

Verify that text completion responses now include the created timestamp field when it is non-zero.

Screenshots/Recordings

N/A

Breaking changes

• Yes
• No

Related issues

N/A

Security considerations

None.

Checklist

• I read docs/contributing/README.md and followed the guidelines
• I added/updated tests where appropriate
• I updated documentation where needed
• I verified builds succeed (Go and UI)
• I verified the CI pipeline passes locally if applicable

[coderabbitai]

📝 Walkthrough

Walkthrough

Text completion responses now expose optional creation timestamps, with OpenAI and Replicate streaming paths preserving timestamps through emitted chunks and schema conversions.

Changes

Text completion timestamp propagation

Layer / File(s)	Summary
Update text completion schema contracts core/schemas/textcompletions.go	Adds the optional Created Unix timestamp to responses and changes ExtraParams to map[string]any while preserving its excluded JSON behavior.
Preserve timestamps in response conversions core/schemas/mux.go, core/providers/replicate/text.go	Populates Created when converting chat responses and Replicate responses to text completion responses.
Propagate timestamps through streaming responses core/providers/utils/utils.go, core/providers/openai/openai.go, core/providers/replicate/replicate.go	Captures or parses provider timestamps and assigns them to streamed and terminal completion chunks.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: bearts, tejasghatte, akshaydeo

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title is concise and matches the main change: adding created timestamps for text completions, especially for OpenAI.
Description check	✅ Passed	The description covers the required template sections and is mostly complete, including summary, changes, testing, and checklist items.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch 07-21-fix_adds_created_timestamp_for_openai_text_completions

---

_{Comment @coderabbitai help to get the list of available commands.}

[sammaji]

• fix: adds created timestamp for openai text completions #5407 👈 (View in Graphite)
• chore: adds docs for azure model router #5175
• dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[greptile-apps]

Confidence Score: 5/5

This looks safe to merge.

• The previously missing timestamp is now propagated through the identified conversion and streaming paths.
• All callers of the updated response helper use the new signature.
• No blocking issue was found in the changed code.

Important Files Changed

Filename	Overview
core/providers/openai/openai.go	Captures the stream creation timestamp and includes it in the final text completion response.
core/providers/replicate/replicate.go	Adds the prediction creation timestamp to Replicate text completion stream responses.
core/providers/replicate/text.go	Maps the Replicate prediction timestamp into unary text completion responses.
core/providers/utils/utils.go	Extends the text completion response helper to accept and preserve a creation timestamp.
core/schemas/mux.go	Copies creation timestamps through every chat-to-text conversion branch.
core/schemas/textcompletions.go	Adds the created response field and updates the extra-parameter map alias.

_{Reviews (3): Last reviewed commit: "fix: adds created timestamp for openai t..." | Re-trigger Greptile}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@core/schemas/textcompletions.go`:
- Line 70: Update the created field in the text-completion response schema to
remove omitempty so the required Unix timestamp is always serialized, preserving
the existing json field name and type.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: d807a1df-2734-4f47-b936-30ed1871da03

📥 Commits

Reviewing files that changed from the base of the PR and between cba1d0f and 5ce7f0b.

📒 Files selected for processing (1)

• core/schemas/textcompletions.go

[coderabbitai]

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

core/providers/openai/openai.go (1)

697-715: 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win

Capture Created before skipping empty-choice chunks.

The handler supports usage-only chunks, but it skips every response with no choices before the new capture block. If such a chunk carries the only non-zero Created value, the final chunk omits the timestamp. Move the capture above the empty-choice check and remove the duplicate block.

Proposed fix

+			if response.Created != 0 && created == 0 {
+				created = response.Created
+			}
+
 			// Skip empty responses or responses without choices
 			if len(response.Choices) == 0 {
 				continue
 			}
-
-			if response.Created != 0 && created == 0 {
-				created = response.Created
-			}

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@core/providers/openai/openai.go` around lines 697 - 715, Move the
response.Created capture logic in the streaming handler before the
len(response.Choices) == 0 early-continue, so usage-only chunks can populate
created; remove the later duplicate capture while leaving messageID and
choice/finish-reason handling unchanged.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Outside diff comments:
In `@core/providers/openai/openai.go`:
- Around line 697-715: Move the response.Created capture logic in the streaming
handler before the len(response.Choices) == 0 early-continue, so usage-only
chunks can populate created; remove the later duplicate capture while leaving
messageID and choice/finish-reason handling unchanged.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 33b09e60-5b89-4daf-860a-0b40e5bee824

📥 Commits

Reviewing files that changed from the base of the PR and between 5ce7f0b and 222cb4f.

📒 Files selected for processing (6)

• core/providers/openai/openai.go
• core/providers/replicate/replicate.go
• core/providers/replicate/text.go
• core/providers/utils/utils.go
• core/schemas/mux.go
• core/schemas/textcompletions.go

🚧 Files skipped from review as they are similar to previous changes (1)

• core/schemas/textcompletions.go

[akshaydeo]

Merge activity

• Jul 21, 2:59 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• Jul 21, 3:00 PM UTC: @akshaydeo merged this pull request with Graphite.

The base branch was changed.