feat:desinger support upload to OBS #170
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,24 +1,34 @@ | ||
| import { mergeConfig, loadEnv } from 'vite'; | ||
| import eslint from 'vite-plugin-eslint'; | ||
| import baseConfig from './vite.config.base'; | ||
| import { resolve } from 'path'; | ||
| import { configDotenv } from 'dotenv'; | ||
|
|
||
| // 加载 dev.env 文件 | ||
| configDotenv({ | ||
| path: resolve(__dirname, '../dev.env'), | ||
| }); | ||
|
|
||
| // 加载环境变量（development 模式会读取 .env.development 和 .env） | ||
| const env = loadEnv('development', process.cwd()); | ||
|
|
||
| const proxyConfig = { | ||
| [env.VITE_BASE_API]: { | ||
| target: env.VITE_SERVER_HOST, | ||
| changeOrigin: true, | ||
| logLevel: 'debug', | ||
| rewrite: (path: string) => | ||
| path.replace( | ||
| new RegExp(`${env.VITE_BASE_API}`), | ||
| '', | ||
| ), | ||
| }, | ||
| [env.VITE_MOCK_SERVER_HOST]: { | ||
| target: env.VITE_SERVER_HOST, | ||
| changeOrigin: true, | ||
| rewrite: (path: string) => { | ||
|
|
||
| return path.replace(new RegExp(`${env.VITE_MOCK_SERVER_HOST}`), '/mock') | ||
| }, | ||
| }, | ||
| }; | ||
|
Comment on lines
15
to
34
There was a problem hiding this comment. 🧩 Analysis chain🏁 Script executed: find . -name "vite.config.dev.ts" -o -name "vite.config.ts" | head -20Repository: opentiny/tiny-pro Length of output: 105 🏁 Script executed: # Find the exact file and read it
fd -t f "vite.config" | grep -E "dev|vite" | head -10Repository: opentiny/tiny-pro Length of output: 387 🏁 Script executed: # List files in template/tinyvue/config directory
ls -la template/tinyvue/config/ 2>/dev/null || find . -path "*/template/tinyvue/config*" -type dRepository: opentiny/tiny-pro Length of output: 719 🏁 Script executed: cat -n template/tinyvue/config/vite.config.dev.tsRepository: opentiny/tiny-pro Length of output: 2138 🏁 Script executed: # Also check the base config to understand env setup
cat -n template/tinyvue/config/vite.config.base.tsRepository: opentiny/tiny-pro Length of output: 2445 🏁 Script executed: # Check for env files to see what patterns are used
find . -name ".env*" -o -name "dev.env" | head -20Repository: opentiny/tiny-pro Length of output: 392 🏁 Script executed: # Look for the dev.env file mentioned in the code
cat template/tinyvue/dev.env 2>/dev/null || echo "File not found"Repository: opentiny/tiny-pro Length of output: 478 🏁 Script executed: # Check if there's a .env.development file
cat template/tinyvue/.env.development 2>/dev/null || echo "File not found"Repository: opentiny/tiny-pro Length of output: 75 🏁 Script executed: # Search for any .env files in template/tinyvue
find template/tinyvue -name ".env*" -o -name "*env*" | grep -E "\.(env|example)" | head -10Repository: opentiny/tiny-pro Length of output: 135 🏁 Script executed: # Check for other usages of similar RegExp patterns in the codebase
rg "new RegExp" template/tinyvue/config/ -A 2 -B 2Repository: opentiny/tiny-pro Length of output: 734 🏁 Script executed: # Verify that the env values are only in configuration files and not loaded from user input
rg "VITE_BASE_API|VITE_MOCK_SERVER_HOST" template/tinyvue/ --type ts --type js | head -20Repository: opentiny/tiny-pro Length of output: 2104 🏁 Script executed: # Check if env variables can be overridden at runtime
git log --oneline -5 -- template/tinyvue/dev.env template/tinyvue/.env 2>/dev/null | head -10Repository: opentiny/tiny-pro Length of output: 86 🏁 Script executed: # Double-check if there are any dynamic sources that could feed into these env values
rg "VITE_BASE_API|VITE_MOCK_SERVER_HOST" template/tinyvue -B 3 -A 1 --type ts | grep -E "(process\.env|process\.argv|require\(|import)" | head -20Repository: opentiny/tiny-pro Length of output: 2186 🏁 Script executed: # Check if loadEnv can load from user input or just from .env files
rg "loadEnv" template/tinyvue --type ts -A 2 -B 2Repository: opentiny/tiny-pro Length of output: 2295 Simplify path replacement to improve code clarity. The proxy configuration uses Consider using direct string replacement instead of RegExp on lines 21-24 and 31: Suggested improvements// Line 21-24: Instead of
rewrite: (path: string) =>
path.replace(
new RegExp(`${env.VITE_BASE_API}`),
'',
),
// Use
rewrite: (path: string) => path.replace(env.VITE_BASE_API, ''),// Line 31: Instead of
return path.replace(new RegExp(`${env.VITE_MOCK_SERVER_HOST}`), '/mock')
// Use
return path.replace(env.VITE_MOCK_SERVER_HOST, '/mock')Both approaches replace only the first occurrence, so they're functionally equivalent. 🧰 Tools🪛 ast-grep (0.40.3)[warning] 21-21: Regular expression constructed from variable input detected. This can lead to Regular Expression Denial of Service (ReDoS) attacks if the variable contains malicious patterns. Use libraries like 'recheck' to validate regex safety or use static patterns. (regexp-from-variable) [warning] 30-30: Regular expression constructed from variable input detected. This can lead to Regular Expression Denial of Service (ReDoS) attacks if the variable contains malicious patterns. Use libraries like 'recheck' to validate regex safety or use static patterns. (regexp-from-variable) 🤖 Prompt for AI Agents |
||
|
|
@@ -34,6 +44,16 @@ export default mergeConfig( | |
| ...proxyConfig, | ||
| }, | ||
| }, | ||
| define: { | ||
| // 确保 VITE_LOWCODE_DESIGNER_ENABLED 被注入到客户端代码 | ||
| 'import.meta.env.VITE_LOWCODE_DESIGNER_ENABLED': JSON.stringify( | ||
| process.env.VITE_LOWCODE_DESIGNER_ENABLED || 'false' | ||
| ), | ||
| // 确保 VITE_LOWCODE_DESIGNER_URL 被注入到客户端代码 | ||
| 'import.meta.env.VITE_LOWCODE_DESIGNER_URL': JSON.stringify( | ||
| process.env.VITE_LOWCODE_DESIGNER_URL || 'http://localhost:8090' | ||
| ), | ||
| }, | ||
| plugins: [ | ||
| eslint({ | ||
| include: ['src/**/*.ts', 'src/**/*.tsx', 'src/**/*.vue'], | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🏁 Script executed:
Repository: opentiny/tiny-pro
Length of output: 2046
🏁 Script executed:
rg "VITE_LOWCODE_DESIGNER_URL" --type yaml --type javascript --type typescriptRepository: opentiny/tiny-pro
Length of output: 95
🏁 Script executed:
Repository: opentiny/tiny-pro
Length of output: 5076
🏁 Script executed:
rg "VITE_LOWCODE_DESIGNER" -A 2 -B 2Repository: opentiny/tiny-pro
Length of output: 3636
🏁 Script executed:
rg "ai.opentiny.design/designer" -A 2 -B 2Repository: opentiny/tiny-pro
Length of output: 43
🏁 Script executed:
Repository: opentiny/tiny-pro
Length of output: 43
Update
VITE_LOWCODE_DESIGNER_URLto use the locally deployed designer path.Line 67's comment correctly states to use relative path
/designerafter deployment, but line 68 sets an absolute external URL. The production build config invite.config.prod.tsdefaults to/designer, and the workflow explicitly builds and deploys the designer locally (lines 69-81). Change line 68 to:🤖 Prompt for AI Agents