chore: upgrade v1.17

[hwbrzzl]

📑 Description

@coderabbitai summary

✅ Checks

• Added test cases for my code

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Process dependency is propagated into Docker and SQL Server initialization: Docker gains a process field and constructor parameter; SQL Server accepts and stores a process and validates it before creating Docker resources; tests and image driver creation are updated to pass the process through.

Changes

Cohort / File(s)	Summary
Docker Container Setup docker.go, docker_test.go	Added process contractsprocess.Process field to Docker, changed NewDocker signature to accept process, stored it on the struct, passed process into image driver creation and tests updated to pass process.New() to constructor.
SQL Server Integration service_provider.go, sqlserver.go	Updated NewSqlserver signature to accept process and added process field to Sqlserver. Docker() now checks for non-nil process and passes it into NewDocker. service_provider.go call updated to app.MakeProcess().
Image Driver ...testingdocker.NewImageDriver(...) usages	Image driver creation calls now include the process parameter (e.g., testingdocker.NewImageDriver(image, r.process)).
Module dependencies go.mod	Bumped goravel/framework pseudo-version and refreshed indirect dependencies (Charmbracelet modules, muesli, updated x/* packages, grpc v1.78.0, mapstructure/v2 v2.5.0, etc.).

Sequence Diagram(s)

sequenceDiagram
    participant SP as ServiceProvider
    participant AppProc as App Process (app.MakeProcess)
    participant Sql as Sqlserver
    participant Docker as Docker
    participant Image as ImageDriver

    SP->>AppProc: request process
    AppProc-->>SP: process instance
    SP->>Sql: NewSqlserver(config, log, process, conn)
    Sql->>Sql: store process
    Sql->>Docker: Docker() // create container
    Docker->>Docker: validate process != nil
    Docker->>Image: NewImageDriver(image, process)
    Image-->>Docker: image driver with process
    Docker-->>Sql: return container handle

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch bowen/upgrade-v1.17

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[Copilot]

Pull request overview

Updates the sqlserver driver to align with the newer Goravel framework version by wiring in the Process facade where required for Docker-based testing utilities.

Changes:

• Bumped github.com/goravel/framework dependency and refreshed transitive module versions (go.mod / go.sum).
• Injected process.Process into Sqlserver and Docker so Docker image/container helpers can use the framework process implementation.
• Updated the service provider and Docker test suite to supply the Process dependency.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
sqlserver.go	Adds process.Process to the driver and passes it through when constructing Docker driver.
service_provider.go	Updates driver construction to pass app.MakeProcess().
docker.go	Extends Docker driver to carry/process-inject contracts/process.Process into image driver creation.
docker_test.go	Updates tests to pass a concrete framework process implementation.
go.mod	Upgrades framework version and refreshes indirect dependencies.
go.sum	Updates module checksums to match the upgraded dependency graph.

---

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