Skip to content

Commit 7a269c8

Browse files
committed
refactor: remove deprecated tools/file_operations_component
The Rust-based file_operations_component has been superseded by: - tools/file_ops (Go implementation) - current production path - tools/file_ops_external (WASM component with AOT) - preferred path Changes: - Delete tools/file_operations_component directory (Rust CLI) - Move WIT interface definition to tools/file_ops/wit/ - Update all toolchain references to use //tools/file_ops:wit_files - Update dual_implementation tests to use Go implementation - Remove dependabot entry for deleted Cargo.toml This removes ~600 lines of unused Rust code and simplifies the file operations toolchain architecture.
1 parent 25b99b7 commit 7a269c8

File tree

10 files changed

+19
-621
lines changed

10 files changed

+19
-621
lines changed

.github/dependabot.yml

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -81,26 +81,6 @@ updates:
8181
- "automated"
8282
- "performance"
8383

84-
- package-ecosystem: "cargo"
85-
directory: "/tools/file_operations_component"
86-
schedule:
87-
interval: "weekly"
88-
day: "monday"
89-
time: "06:00"
90-
open-pull-requests-limit: 10
91-
reviewers:
92-
- "avrabe"
93-
assignees:
94-
- "avrabe"
95-
commit-message:
96-
prefix: "rust/file-ops"
97-
include: "scope"
98-
labels:
99-
- "dependencies"
100-
- "rust"
101-
- "automated"
102-
- "wasm"
103-
10484
- package-ecosystem: "cargo"
10585
directory: "/tools-builder/toolchains"
10686
schedule:

test/dual_implementation/BUILD.bazel

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,28 +6,29 @@ load("//toolchains:dual_file_ops_toolchain.bzl", "dual_file_ops_toolchain")
66
package(default_visibility = ["//visibility:public"])
77

88
# Test dual toolchain with actual components
9+
# Uses Go-based file_ops for both implementations (production pattern)
910
dual_file_ops_toolchain(
1011
name = "test_dual_toolchain",
1112
implementation_preference = "auto",
12-
rust_component = "//tools/file_operations_component:file_ops", # Actual Rust implementation
13-
tinygo_component = "//tools/file_operations_component:file_ops", # Current Rust implementation as mock TinyGo
14-
wit_files = ["//tools/file_operations_component:wit_files"],
13+
rust_component = "//tools/file_ops:file_ops",
14+
tinygo_component = "//tools/file_ops:file_ops",
15+
wit_files = ["//tools/file_ops:wit_files"],
1516
)
1617

1718
# Test security-focused configuration
1819
dual_file_ops_toolchain(
1920
name = "test_security_toolchain",
2021
implementation_preference = "tinygo",
21-
tinygo_component = "//tools/file_operations_component:file_ops",
22-
wit_files = ["//tools/file_operations_component:wit_files"],
22+
tinygo_component = "//tools/file_ops:file_ops",
23+
wit_files = ["//tools/file_ops:wit_files"],
2324
)
2425

2526
# Test performance-focused configuration
2627
dual_file_ops_toolchain(
2728
name = "test_performance_toolchain",
2829
implementation_preference = "rust",
29-
rust_component = "//tools/file_operations_component:file_ops",
30-
wit_files = ["//tools/file_operations_component:wit_files"],
30+
rust_component = "//tools/file_ops:file_ops",
31+
wit_files = ["//tools/file_ops:wit_files"],
3132
)
3233

3334
# Build tests to verify toolchain configurations

toolchains/BUILD.bazel

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -199,7 +199,7 @@ bzl_library(
199199
file_ops_toolchain(
200200
name = "file_ops_toolchain_impl",
201201
file_ops_component = "//tools/file_ops:file_ops",
202-
wit_files = ["//tools/file_operations_component:wit_files"],
202+
wit_files = ["//tools/file_ops:wit_files"],
203203
)
204204

205205
toolchain(
@@ -215,7 +215,7 @@ toolchain(
215215
file_ops_toolchain(
216216
name = "file_ops_toolchain_external_impl",
217217
file_ops_component = "//tools/file_ops_external:file_ops_external",
218-
wit_files = ["//tools/file_operations_component:wit_files"],
218+
wit_files = ["//tools/file_ops:wit_files"],
219219
)
220220

221221
toolchain(

toolchains/file_ops_toolchain.bzl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ load("@rules_wasm_component//toolchains:file_ops_toolchain.bzl", "file_ops_toolc
4343
file_ops_toolchain(
4444
name = "file_ops_toolchain_impl",
4545
file_ops_component = "@rules_wasm_component//tools/file_ops:file_ops",
46-
wit_files = ["@rules_wasm_component//tools/file_operations_component:wit_files"],
46+
wit_files = ["@rules_wasm_component//tools/file_ops:wit_files"],
4747
visibility = ["//visibility:public"],
4848
)
4949

tools/file_operations_component/BUILD.bazel

Lines changed: 0 additions & 54 deletions
This file was deleted.

tools/file_operations_component/Cargo.toml

Lines changed: 0 additions & 26 deletions
This file was deleted.

0 commit comments

Comments
 (0)