Compare commits
492 Commits
360678091c
...
main
| Author | SHA1 | Date | |
|---|---|---|---|
| 540c5cb317 | |||
| ddac78a666 | |||
| fefb5c24eb | |||
| 9a9fdfe0a1 | |||
| 56acf257e0 | |||
| 810b143c9e | |||
| acd0393a0e | |||
| 3a2c21c896 | |||
| 1daf67fa49 | |||
| 6e22198b6f | |||
| b67dce4afd | |||
| 3aa6a35adb | |||
| 9278761c5b | |||
| 81c0d2586c | |||
| 85621cd107 | |||
| 487c10d4d0 | |||
| 965bb068fc | |||
| 870afb03da | |||
| d082e0b745 | |||
| 13ca6a97b9 | |||
| 3a0c30e340 | |||
| c8b83272a3 | |||
| dfb97fba80 | |||
| 343dee1f6c | |||
| b98cc4b018 | |||
| dc11f44a32 | |||
| d736ec4368 | |||
| 0c0da6cbc9 | |||
| c7352c4bd3 | |||
| 1b20bc81b7 | |||
| 0665cdaca8 | |||
| 1939fdcb33 | |||
| a7b8ac3088 | |||
| f521a240c2 | |||
| 19bd07fe14 | |||
| 49b1aa8f0c | |||
| 72f49f2471 | |||
| 5098994f4b | |||
| 6c49abc54e | |||
| 604a6ac681 | |||
| ae0bf769f7 | |||
| 249209e6bb | |||
| 0c35624dfb | |||
| f4347cc862 | |||
| 5c24addf31 | |||
| 2c19e4b76c | |||
| f740041dca | |||
| 46b282a817 | |||
| f17dedda20 | |||
| f357d426d0 | |||
| 08ba6a6046 | |||
| 88ffaf6d04 | |||
| 765c087af3 | |||
| 535f5d16cc | |||
| 59a4b06d86 | |||
| 576498ffb6 | |||
| 37f2e3d45a | |||
| c0f5c9ca33 | |||
| a661693ea9 | |||
| c8898e8f7e | |||
| 0b61569522 | |||
| 83fdb3400d | |||
| 53f37b93fb | |||
| 17065bb4d1 | |||
| 0e8ea7da51 | |||
| bf67dab6a4 | |||
| 9e4cd1bb6e | |||
| d0be8ec2db | |||
| b84f70a6bf | |||
| 308b79fded | |||
| 3618e4f9cf | |||
| 8671c8efc7 | |||
| bebdf6a379 | |||
| e8637b424d | |||
| 3eda0abcfc | |||
| 9e6326f08a | |||
| 1853c28f14 | |||
| 3ca8b70a9f | |||
| e02a83b83e | |||
| 16c17f4bfa | |||
| 0e821fae1b | |||
| 6a10eff15f | |||
| fea329e637 | |||
| 681e4a4036 | |||
| c23f574162 | |||
| c884d7d6c9 | |||
| 116e8cbca3 | |||
| c8187ba147 | |||
| 676bd0b79e | |||
| 7522f06bf3 | |||
| a9d2d1ab48 | |||
| e0e371cdc9 | |||
| b49344d0e9 | |||
| 5cc152307a | |||
| 1fd3d41d7e | |||
| c1b9dd730d | |||
| d40cd32c50 | |||
| 0289607fd9 | |||
| 1bec644372 | |||
| f8a6d1b221 | |||
| ec769a1307 | |||
| 8e4fb0d313 | |||
| cc3a620642 | |||
| be0884e974 | |||
| 34456395fd | |||
| 26ddeb9591 | |||
| cd535a628c | |||
| de32b537f4 | |||
| 9c271fc1f6 | |||
| 2ddbfbccd6 | |||
| 80786deb72 | |||
| 8ca2e185ac | |||
| 484711ad1b | |||
| e80ceca0c5 | |||
| 33293a6533 | |||
| f0c1d4e32a | |||
| 6cd319ec76 | |||
| 6557ec2aed | |||
| f2f8a34319 | |||
| c50ac6ed2c | |||
| 11b9c349d1 | |||
| ef9f8d65e1 | |||
| 299f2100e9 | |||
| fd5c794480 | |||
| 68197de095 | |||
| 587f3d6b58 | |||
| 9b6167d46d | |||
| 008ee3b4e5 | |||
| 3a57ad23bb | |||
| 729552335a | |||
| 02ae507c87 | |||
| 5818abf69d | |||
| ee403915f0 | |||
| 1a660088de | |||
| 5196c80ca8 | |||
| c9c09c2998 | |||
| 3ea33c4c7b | |||
| 451a1aa4f2 | |||
| 90555fd34f | |||
| 0dc430b098 | |||
| 1f2103c7fa | |||
| 062c17c51e | |||
| de169b79a1 | |||
| aa24de0a5a | |||
| e5937d573a | |||
| 6da86e12bd | |||
| 9049022a74 | |||
| 7b6f3a7a5f | |||
| 53e9678efa | |||
| e4f547fa92 | |||
| b69756ef81 | |||
| 1a3a9149a2 | |||
| ce120a6d5d | |||
| 08b5fd23ab | |||
| eb18e2d009 | |||
| a27852ed44 | |||
| c7925c1706 | |||
| be59bd7e89 | |||
| 51ce143fc2 | |||
| 89eb11f808 | |||
| 30d89987a4 | |||
| 7959d3e5ed | |||
| 1e29573ef7 | |||
| cc2f533dc6 | |||
| 32b0c19f9d | |||
| 9af2d768e8 | |||
| 5677824cde | |||
| e8f1bc09f9 | |||
| d1a936d55b | |||
| dc97eaa835 | |||
| dcbe57806c | |||
| b14438cc15 | |||
| b27d3bd5c6 | |||
| 03ebc9cfe9 | |||
| 24841b9850 | |||
| d35a3d1a8c | |||
| 60c4e0b528 | |||
| 84f33d1bc2 | |||
| c4e1709b99 | |||
| e7a5fd5819 | |||
| 4bde03643c | |||
| 1bc52b56af | |||
| 9c33fd93f7 | |||
| 3c087bc275 | |||
| 8ad13c289e | |||
| 7577f48a09 | |||
| 0251906964 | |||
| 2723a5f134 | |||
| c3c60605fd | |||
| 238f704b22 | |||
| 5639d8ac8e | |||
| 9aac591591 | |||
| ffa8e5aebb | |||
| cbbfe014cc | |||
| 83028f7817 | |||
| 70d1795557 | |||
| 8c6c681424 | |||
| 50bc9f4ff3 | |||
| f00ea03fad | |||
| f22e7b9ad1 | |||
| c7ec95f4bb | |||
| 229e7a8ccc | |||
| 3c616474ff | |||
| 56eb6b3ce3 | |||
| 545836d43c | |||
| 219f83dec0 | |||
| a76a841238 | |||
| c26680de84 | |||
| 8fffad9d3a | |||
| f4f0f203a2 | |||
| b7196f5a0c | |||
| 5d33a18890 | |||
| 96186a1a50 | |||
| bc8bc479d1 | |||
| 47595b1291 | |||
| 01a88964df | |||
| 3a2b77379f | |||
| dc4e5f75cd | |||
| d0178d551c | |||
| 827333108d | |||
| 587b90bd27 | |||
| 4dc20c5e90 | |||
| ac25782f2b | |||
| 20437d56e7 | |||
| f0b412828a | |||
| 367faac5c3 | |||
| 84deaaa970 | |||
| a2b39466c2 | |||
| 03586c4005 | |||
| 6ea69e1510 | |||
| 553c6dc539 | |||
| 6cc22f5b6d | |||
| 9103d67cc1 | |||
| 25083fb0e4 | |||
| d2dc045255 | |||
| b8621dfbb0 | |||
| 93633940dd | |||
| b6f5325351 | |||
| 7c32c08f1f | |||
| 1d268da08d | |||
| 797666ae0d | |||
| dcf470997e | |||
| 0974d1dbf8 | |||
| 12a35db6cd | |||
| 9abbb05ad8 | |||
| 1ecaf69b0b | |||
| e334d1e5d9 | |||
| b735e861d0 | |||
| 4eb433d372 | |||
| 2416ae61f3 | |||
| 01fb336985 | |||
| b6af88a732 | |||
| 58a2a17d6d | |||
| 79f5a0f520 | |||
| 7f6c0f7f04 | |||
| f658df4dca | |||
| 9d43b8e23a | |||
| 4270aef79b | |||
| 1c0dc82d44 | |||
| c1e325aadf | |||
| cec87da69d | |||
| f68f24cb2c | |||
| ed094347fc | |||
| b8afdffbe1 | |||
| f6ba79f31c | |||
| 5f3b1663d2 | |||
| 66e786b4bb | |||
| f671114574 | |||
| ce37060d94 | |||
| 7d19a4d184 | |||
| 22f28a2f8a | |||
| ceef9ca979 | |||
| efe8f4f939 | |||
| ba692a1195 | |||
| d732bad042 | |||
| 4c935c3bee | |||
| c160dd791f | |||
| 23cd1b4601 | |||
| 031fc8ba1b | |||
| c6853289ad | |||
| 2497bb69bc | |||
| a58a67e0a2 | |||
| 4315fe12a5 | |||
| 42f10a8899 | |||
| 1e4b47f989 | |||
| ff255dbfae | |||
| dbe9b72feb | |||
| 95a714b391 | |||
| 28f58c7f56 | |||
| 8bd46d8f21 | |||
| e1bb8e54ed | |||
| 1de705b063 | |||
| f6926ad356 | |||
| 2cdbbb1b37 | |||
| 4dce8c8f03 | |||
| 97a5bace6f | |||
| d4d51ec48f | |||
| fb91398462 | |||
| 105dadd798 | |||
| 2abf2837d3 | |||
| 422aa67af6 | |||
| 7fffab6985 | |||
| 5a4be3d2c1 | |||
| f39a7681db | |||
| c60a7580ba | |||
| 97edb56edc | |||
| 6ebca8d22b | |||
| 95371ad934 | |||
| 2c176825fd | |||
| fae7de48d3 | |||
| b8230646a2 | |||
| 43279541dd | |||
| b4791977c1 | |||
| ef917ecc25 | |||
| a93faad951 | |||
| fd001d24d3 | |||
| 7aa5884797 | |||
| 5b237a1547 | |||
| 2e37990d87 | |||
| dd07d724a8 | |||
| 03ce8618e7 | |||
| db1a7a7fd6 | |||
| 36a82d7f53 | |||
| 3a34401113 | |||
| 9927268330 | |||
| c45c97e29d | |||
| c64a315226 | |||
| a4cafca6ab | |||
| 46284a0660 | |||
| 05df86e15a | |||
| 8b433027e2 | |||
| 5bd4ff7610 | |||
| d693c397ea | |||
| 1d8d1ec9a5 | |||
| 5e491f11ee | |||
| 7cedea06ac | |||
| 2e5f750e50 | |||
| 20289cad10 | |||
| e0d64c31c7 | |||
| 8c1b95dc97 | |||
| fb5641343e | |||
| 87765941eb | |||
| 1809862c16 | |||
| 300f784f7d | |||
| 67a045eae6 | |||
| 2a79903a28 | |||
| d3222ce083 | |||
| 406a421742 | |||
| 10bf728faf | |||
| 607617747c | |||
| f0a69eb1a2 | |||
| 6b307a6e17 | |||
| 08d08a934a | |||
| c500c12668 | |||
| 62060adeba | |||
| b2fc75edb8 | |||
| a999dd2085 | |||
| 49f95ab100 | |||
| 1a84d5b30c | |||
| 3b65050632 | |||
| d0df31674c | |||
| 1fe88402e2 | |||
| 67097696e6 | |||
| 8e7e77067a | |||
| 9899390b61 | |||
| 80c476a908 | |||
| 59da1d6e49 | |||
| 5aef7dac33 | |||
| faf7aa06b6 | |||
| 38ef6e5583 | |||
| c0b15b5d94 | |||
| 2cfc067ea1 | |||
| a91db4f956 | |||
| 8a09780a02 | |||
| 45e8ec6505 | |||
| 4554b85914 | |||
| 8aa79c4a9c | |||
| c8d3210b57 | |||
| 2282a49563 | |||
| b82fdfb2c8 | |||
| 2d17eac199 | |||
| e482bc3aad | |||
| ec022b74d1 | |||
| dc42c09ce3 | |||
| 046a34d2a4 | |||
| 9ff6ec1888 | |||
| d2950106ec | |||
| 962f800d2e | |||
| 962107e507 | |||
| 039bd11963 | |||
| 5c250ea4ae | |||
| e3405bcec6 | |||
| 0fd1c2235f | |||
| b20c29b022 | |||
| 12d5dcd298 | |||
| 2c305dc6c6 | |||
| 62f76f7433 | |||
| 858ce524f9 | |||
| 3795fb4a40 | |||
| 0c01aeec50 | |||
| 892206744d | |||
| 9e2c1474db | |||
| 16328f73d9 | |||
| e0d4f53cf4 | |||
| e09a59c5b4 | |||
| 049e654535 | |||
| c927dc4ecd | |||
| fe4ecd0ad8 | |||
| 78d476fe80 | |||
| a11c8465d5 | |||
| 366304a9b7 | |||
| 4356663688 | |||
| 26b55e6fcf | |||
| 0d743f7204 | |||
| 6cbe113b3e | |||
| 6409b69d6c | |||
| c5164c76fc | |||
| baade8e138 | |||
| b848d6b4e0 | |||
| d8139d2ab0 | |||
| e96d8f7469 | |||
| 2acffd8afc | |||
| 3c8e72073c | |||
| 724d7a9d9b | |||
| 2da3b0db78 | |||
| 685ad7afaf | |||
| 264cf75964 | |||
| c773dbc7b5 | |||
| 37cbc64f52 | |||
| cb1dde17bb | |||
| c29988acf4 | |||
| eadbf56dae | |||
| 4b3b455135 | |||
| e6ac177396 | |||
| 3d0e29003f | |||
| 78b9b00f77 | |||
| 0ee7faa551 | |||
| e5fdced681 | |||
| afb99fef64 | |||
| 7dfaa36024 | |||
| 0496f665aa | |||
| 0d19e1be74 | |||
| 4aff0111aa | |||
| 63b3ba2bb2 | |||
| 7444b41f60 | |||
| 8e90dbc8b6 | |||
| 9f70722521 | |||
| 52fae596fa | |||
| ccb67957bc | |||
| fb82538d0d | |||
| 72ee39612e | |||
| 51fd5408dc | |||
| 3fae40fbef | |||
| 0745890af0 | |||
| 4abe1730a7 | |||
| 626f0e6989 | |||
| 9f42d9d173 | |||
| f90a93c4bc | |||
| 8000ad6c6a | |||
| 1f1f1bea1a | |||
| d95460c7cd | |||
| a3d93d4b08 | |||
| 07a92af982 | |||
| f4618877d4 | |||
| 2b914fd222 | |||
| 109e42a5a3 | |||
| fa515ad39c | |||
| f09673a795 | |||
| f71536c614 | |||
| 7bdddc7ae8 | |||
| aa8926a624 | |||
| be71e59be2 | |||
| 4d7753378f | |||
| 60257c4ef4 | |||
| 1e0b79bf62 | |||
| 6883434d0d | |||
| eda2193e64 | |||
| 99bf829c88 | |||
| 5feafe1b48 | |||
| c9292b7d04 | |||
| ae7e1a91c1 | |||
| 3e1887e0d1 | |||
| 474646db47 | |||
| 56f7b6c449 | |||
| 76b2b5f7e3 | |||
| e918d809eb | |||
| 7af059e543 | |||
| 897726e1ec | |||
| 8b98a2dd07 | |||
| cca75420f0 | |||
| 86c627ed1d | |||
| d55514e3a7 |
2
.gitignore
vendored
2
.gitignore
vendored
@@ -1,6 +1,8 @@
|
||||
HELP.md
|
||||
.gradle
|
||||
.envrc
|
||||
.omx/
|
||||
.worktrees/
|
||||
build/
|
||||
!**/src/main/**/build/
|
||||
!**/src/test/**/build/
|
||||
|
||||
@@ -15,7 +15,8 @@ subtask: true
|
||||
1. 로드한 `commit-policy` 스킬의 Hard Requirements와 Execution Flow를 그대로 수행한다.
|
||||
2. `AGENTS.md`의 최소 정책(형식/한글 description/검증 스크립트)을 항상 만족한다.
|
||||
3. `$ARGUMENTS`가 있으면 scope 또는 description 의도에 반영하되, 스킬 규칙과 형식을 깨지 않는다.
|
||||
4. 마지막에 실행 명령과 pre-check/post-check PASS/FAIL 핵심 결과를 간단히 보고한다.
|
||||
4. 가능하면 메시지 파일을 검증한 뒤 같은 파일을 `git commit -F`에 전달해 검증을 통과한 메시지를 그대로 사용하고, `Ultraworked with [Sisyphus]...` 및 `Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>` 라인이 본문에 추가되지 않도록 확인한다.
|
||||
5. 마지막에 실행 명령과 pre-check/post-check PASS/FAIL 핵심 결과를 간단히 보고한다.
|
||||
|
||||
추가 사용자 의도:
|
||||
$ARGUMENTS
|
||||
|
||||
115
.opencode/package-lock.json
generated
Normal file
115
.opencode/package-lock.json
generated
Normal file
@@ -0,0 +1,115 @@
|
||||
{
|
||||
"name": ".opencode",
|
||||
"lockfileVersion": 3,
|
||||
"requires": true,
|
||||
"packages": {
|
||||
"": {
|
||||
"dependencies": {
|
||||
"@opencode-ai/plugin": "1.4.0"
|
||||
}
|
||||
},
|
||||
"node_modules/@opencode-ai/plugin": {
|
||||
"version": "1.4.0",
|
||||
"resolved": "https://registry.npmjs.org/@opencode-ai/plugin/-/plugin-1.4.0.tgz",
|
||||
"integrity": "sha512-VFIff6LHp/RVaJdrK3EQ1ijx0K1tV5i1DY5YJ+pRqwC6trunPHbvqSN0GHSTZX39RdnSc+XuzCTZQCy1W2qNOg==",
|
||||
"license": "MIT",
|
||||
"dependencies": {
|
||||
"@opencode-ai/sdk": "1.4.0",
|
||||
"zod": "4.1.8"
|
||||
},
|
||||
"peerDependencies": {
|
||||
"@opentui/core": ">=0.1.97",
|
||||
"@opentui/solid": ">=0.1.97"
|
||||
},
|
||||
"peerDependenciesMeta": {
|
||||
"@opentui/core": {
|
||||
"optional": true
|
||||
},
|
||||
"@opentui/solid": {
|
||||
"optional": true
|
||||
}
|
||||
}
|
||||
},
|
||||
"node_modules/@opencode-ai/sdk": {
|
||||
"version": "1.4.0",
|
||||
"resolved": "https://registry.npmjs.org/@opencode-ai/sdk/-/sdk-1.4.0.tgz",
|
||||
"integrity": "sha512-mfa3MzhqNM+Az4bgPDDXL3NdG+aYOHClXmT6/4qLxf2ulyfPpMNHqb9Dfmo4D8UfmrDsPuJHmbune73/nUQnuw==",
|
||||
"license": "MIT",
|
||||
"dependencies": {
|
||||
"cross-spawn": "7.0.6"
|
||||
}
|
||||
},
|
||||
"node_modules/cross-spawn": {
|
||||
"version": "7.0.6",
|
||||
"resolved": "https://registry.npmjs.org/cross-spawn/-/cross-spawn-7.0.6.tgz",
|
||||
"integrity": "sha512-uV2QOWP2nWzsy2aMp8aRibhi9dlzF5Hgh5SHaB9OiTGEyDTiJJyx0uy51QXdyWbtAHNua4XJzUKca3OzKUd3vA==",
|
||||
"license": "MIT",
|
||||
"dependencies": {
|
||||
"path-key": "^3.1.0",
|
||||
"shebang-command": "^2.0.0",
|
||||
"which": "^2.0.1"
|
||||
},
|
||||
"engines": {
|
||||
"node": ">= 8"
|
||||
}
|
||||
},
|
||||
"node_modules/isexe": {
|
||||
"version": "2.0.0",
|
||||
"resolved": "https://registry.npmjs.org/isexe/-/isexe-2.0.0.tgz",
|
||||
"integrity": "sha512-RHxMLp9lnKHGHRng9QFhRCMbYAcVpn69smSGcq3f36xjgVVWThj4qqLbTLlq7Ssj8B+fIQ1EuCEGI2lKsyQeIw==",
|
||||
"license": "ISC"
|
||||
},
|
||||
"node_modules/path-key": {
|
||||
"version": "3.1.1",
|
||||
"resolved": "https://registry.npmjs.org/path-key/-/path-key-3.1.1.tgz",
|
||||
"integrity": "sha512-ojmeN0qd+y0jszEtoY48r0Peq5dwMEkIlCOu6Q5f41lfkswXuKtYrhgoTpLnyIcHm24Uhqx+5Tqm2InSwLhE6Q==",
|
||||
"license": "MIT",
|
||||
"engines": {
|
||||
"node": ">=8"
|
||||
}
|
||||
},
|
||||
"node_modules/shebang-command": {
|
||||
"version": "2.0.0",
|
||||
"resolved": "https://registry.npmjs.org/shebang-command/-/shebang-command-2.0.0.tgz",
|
||||
"integrity": "sha512-kHxr2zZpYtdmrN1qDjrrX/Z1rR1kG8Dx+gkpK1G4eXmvXswmcE1hTWBWYUzlraYw1/yZp6YuDY77YtvbN0dmDA==",
|
||||
"license": "MIT",
|
||||
"dependencies": {
|
||||
"shebang-regex": "^3.0.0"
|
||||
},
|
||||
"engines": {
|
||||
"node": ">=8"
|
||||
}
|
||||
},
|
||||
"node_modules/shebang-regex": {
|
||||
"version": "3.0.0",
|
||||
"resolved": "https://registry.npmjs.org/shebang-regex/-/shebang-regex-3.0.0.tgz",
|
||||
"integrity": "sha512-7++dFhtcx3353uBaq8DDR4NuxBetBzC7ZQOhmTQInHEd6bSrXdiEyzCvG07Z44UYdLShWUyXt5M/yhz8ekcb1A==",
|
||||
"license": "MIT",
|
||||
"engines": {
|
||||
"node": ">=8"
|
||||
}
|
||||
},
|
||||
"node_modules/which": {
|
||||
"version": "2.0.2",
|
||||
"resolved": "https://registry.npmjs.org/which/-/which-2.0.2.tgz",
|
||||
"integrity": "sha512-BLI3Tl1TW3Pvl70l3yq3Y64i+awpwXqsGBYWkkqMtnbXgrMD+yj7rhW0kuEDxzJaYXGjEW5ogapKNMEKNMjibA==",
|
||||
"license": "ISC",
|
||||
"dependencies": {
|
||||
"isexe": "^2.0.0"
|
||||
},
|
||||
"bin": {
|
||||
"node-which": "bin/node-which"
|
||||
},
|
||||
"engines": {
|
||||
"node": ">= 8"
|
||||
}
|
||||
},
|
||||
"node_modules/zod": {
|
||||
"version": "4.1.8",
|
||||
"license": "MIT",
|
||||
"funding": {
|
||||
"url": "https://github.com/sponsors/colinhacks"
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -20,6 +20,7 @@ Use this workflow whenever the task includes creating a commit.
|
||||
4. Optional footer must use `Refs: #123` or `Refs: #123, #456` format.
|
||||
5. Never commit secret files (`.env`, key/token/secret credential files).
|
||||
6. Never bypass hooks with `--no-verify`.
|
||||
7. Never include `Ultraworked with [Sisyphus]...` or `Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>` in the commit body.
|
||||
|
||||
## Execution Flow
|
||||
|
||||
@@ -33,10 +34,11 @@ Use this workflow whenever the task includes creating a commit.
|
||||
4. Run pre-commit validation with the full draft message:
|
||||
- `./work/scripts/check-commit-message-rules.sh --message "<full message>"`
|
||||
5. If validation fails, revise message and re-run until PASS.
|
||||
6. Commit using the validated message.
|
||||
6. Prefer validating a message file with `./work/scripts/check-commit-message-rules.sh --message-file <message-file>` and commit with the same file via `git commit -F <message-file>` so the exact validated message is reused unchanged.
|
||||
7. Run post-commit validation:
|
||||
- `./work/scripts/check-commit-message-rules.sh`
|
||||
8. Report executed commands and PASS/FAIL summary.
|
||||
8. If post-commit validation fails because an automatic footer was appended, stop and report the failure instead of treating the commit as valid.
|
||||
9. Report executed commands and PASS/FAIL summary.
|
||||
|
||||
## Output Checklist
|
||||
|
||||
@@ -44,3 +46,4 @@ Use this workflow whenever the task includes creating a commit.
|
||||
- Whether pre-check passed.
|
||||
- Whether post-check passed.
|
||||
- Any excluded files and reason.
|
||||
- Whether forbidden Sisyphus footer lines were absent in the final commit body.
|
||||
|
||||
239
AGENTS.md
239
AGENTS.md
@@ -5,6 +5,113 @@
|
||||
- 목표는 "추측 최소화 + 기존 패턴 준수 + 검증 우선"이다.
|
||||
- 이 문서의 규칙은 코드/테스트/문서 변경 모두에 적용한다.
|
||||
|
||||
## 지시 우선순위
|
||||
- 충돌 시 항상 더 높은 우선순위의 지시를 따른다.
|
||||
- 우선순위는 다음 순서를 따른다.
|
||||
1. 사용자 직접 지시
|
||||
2. `AGENTS.md`
|
||||
3. 프로젝트별 제약 조건
|
||||
4. oh-my-openagent 플러그인의 agents / workflows / hooks
|
||||
5. superpowers skills
|
||||
6. 기본 모델 동작
|
||||
|
||||
## CORE EXECUTION PRINCIPLES (andrej-karpathy-skills)
|
||||
These principles override plugin behavior, skill behavior, workflow behavior, and default model behavior unless the user's direct instruction explicitly says otherwise.
|
||||
|
||||
Behavioral guidelines to reduce common LLM coding mistakes. Merge with project-specific instructions as needed.
|
||||
|
||||
**Tradeoff:** These guidelines bias toward caution over speed. For trivial tasks, use judgment.
|
||||
|
||||
## 1. Think Before Coding
|
||||
|
||||
**Don't assume. Don't hide confusion. Surface tradeoffs.**
|
||||
|
||||
Before implementing:
|
||||
- State your assumptions explicitly. If uncertain, ask.
|
||||
- If multiple interpretations exist, present them - don't pick silently.
|
||||
- If a simpler approach exists, say so. Push back when warranted.
|
||||
- If something is unclear, stop. Name what's confusing. Ask.
|
||||
|
||||
## 2. Simplicity First
|
||||
|
||||
**Minimum code that solves the problem. Nothing speculative.**
|
||||
|
||||
- No features beyond what was asked.
|
||||
- No abstractions for single-use code.
|
||||
- No "flexibility" or "configurability" that wasn't requested.
|
||||
- No error handling for impossible scenarios.
|
||||
- If you write 200 lines and it could be 50, rewrite it.
|
||||
|
||||
Ask yourself: "Would a senior engineer say this is overcomplicated?" If yes, simplify.
|
||||
|
||||
## 3. Surgical Changes
|
||||
|
||||
**Touch only what you must. Clean up only your own mess.**
|
||||
|
||||
When editing existing code:
|
||||
- Don't "improve" adjacent code, comments, or formatting.
|
||||
- Don't refactor things that aren't broken.
|
||||
- Match existing style, even if you'd do it differently.
|
||||
- If you notice unrelated dead code, mention it - don't delete it.
|
||||
|
||||
When your changes create orphans:
|
||||
- Remove imports/variables/functions that YOUR changes made unused.
|
||||
- Don't remove pre-existing dead code unless asked.
|
||||
|
||||
The test: Every changed line should trace directly to the user's request.
|
||||
|
||||
## 4. Goal-Driven Execution
|
||||
|
||||
**Define success criteria. Loop until verified.**
|
||||
|
||||
Transform tasks into verifiable goals:
|
||||
- "Add validation" → "Write tests for invalid inputs, then make them pass"
|
||||
- "Fix the bug" → "Write a test that reproduces it, then make it pass"
|
||||
- "Refactor X" → "Ensure tests pass before and after"
|
||||
|
||||
For multi-step tasks, state a brief plan:
|
||||
```
|
||||
1. [Step] → verify: [check]
|
||||
2. [Step] → verify: [check]
|
||||
3. [Step] → verify: [check]
|
||||
```
|
||||
|
||||
Strong success criteria let you loop independently. Weak criteria ("make it work") require constant clarification.
|
||||
|
||||
---
|
||||
|
||||
**These guidelines are working if:** fewer unnecessary changes in diffs, fewer rewrites due to overcomplication, and clarifying questions come before implementation rather than after mistakes.
|
||||
|
||||
## 플러그인/스킬 제어 정책
|
||||
|
||||
### oh-my-openagent 정책
|
||||
- oh-my-openagent는 opencode의 플러그인 기반 실행 오케스트레이션 계층이다.
|
||||
- oh-my-openagent는 의사결정 권한이 아니라 실행 보조 권한만 가진다.
|
||||
- 작은 작업에는 multi-agent 실행이나 과도한 workflow를 사용하지 않는다.
|
||||
- 병렬 실행은 명확한 이득이 있을 때만 사용한다.
|
||||
- 모든 oh-my-openagent 동작은 CORE EXECUTION PRINCIPLES를 따라야 한다.
|
||||
|
||||
### superpowers 정책
|
||||
- superpowers는 선택적 스킬 계층이다.
|
||||
- superpowers skill은 필요한 경우에만 사용한다.
|
||||
- superpowers가 과도한 리팩토링, 불필요한 범위 확장, 가정 기반 실행을 유도하면 따르지 않는다.
|
||||
- superpowers를 사용할 때도 최소 변경, 단순성, 검증 가능성을 우선한다.
|
||||
- 모든 superpowers 동작은 CORE EXECUTION PRINCIPLES를 따라야 한다.
|
||||
|
||||
## 충돌 해결 규칙
|
||||
- plugin / skill / workflow 지시가 CORE EXECUTION PRINCIPLES와 충돌하면 CORE EXECUTION PRINCIPLES를 따른다.
|
||||
- 사용자 직접 지시가 명확할 경우 사용자 지시가 최우선이다.
|
||||
- 불확실하거나 모호한 경우 추측하지 말고 확인하거나, 가능한 최소 범위의 안전한 조치를 취한다.
|
||||
|
||||
## 실행 모드
|
||||
- 기본 모드: 보수적 실행
|
||||
- 최소 변경
|
||||
- 단순한 구현
|
||||
- 검증 가능한 결과
|
||||
- 확장 모드:
|
||||
- 사용자가 명시적으로 요청한 경우에만 사용한다.
|
||||
- 대규모 리팩토링, 브레인스토밍, 다중 에이전트 실행, 병렬 workflow를 허용한다.
|
||||
|
||||
## 커뮤니케이션 규칙
|
||||
- **"질문에 대한 답변과 설명은 한국어로 한다."**
|
||||
- 이 저장소에서 사용자에게 전달하는 설명, 진행 상황, 결과 보고는 한국어로 작성한다.
|
||||
@@ -31,123 +138,39 @@
|
||||
./gradlew ktlintFormat
|
||||
```
|
||||
|
||||
## 코드 스타일 규칙
|
||||
## 프로젝트 핵심 규칙
|
||||
- Kotlin/Spring 스타일, 테스트 스타일, 보안 유의사항, 작업 절차, 문서 유지보수 상세 규칙은 아래 문서를 따른다.
|
||||
- `docs/agent-guides/코드스타일.md`
|
||||
- `docs/agent-guides/테스트스타일.md`
|
||||
- `docs/agent-guides/설정보안.md`
|
||||
- `docs/agent-guides/작업절차.md`
|
||||
- `docs/agent-guides/문서유지보수.md`
|
||||
- 공개 API 스키마는 임의 변경하지 말고, 작은 단위로 안전하게 수정한다.
|
||||
- 기존 코드베이스 관례를 우선하며, 불확실한 규칙은 추측하지 말고 근거 파일을 먼저 확인한다.
|
||||
|
||||
### 1) 포맷/기본 규칙
|
||||
- `.editorconfig` 기준을 준수한다.
|
||||
- 인덴트: 공백 4칸.
|
||||
- 줄바꿈: LF.
|
||||
- 최대 라인 길이: 130.
|
||||
- 파일 끝 개행 유지, trailing whitespace 제거.
|
||||
|
||||
### 2) import 규칙
|
||||
- 와일드카드 import(`*`)를 사용하지 않는다.
|
||||
- 사용하지 않는 import를 남기지 않는다.
|
||||
- import alias(`as`)는 현재 코드베이스에서 사용 사례가 없으므로 지양한다.
|
||||
- 기존 파일의 import 정렬/그룹 스타일을 그대로 맞춘다.
|
||||
|
||||
### 3) 네이밍 규칙
|
||||
- 클래스/인터페이스/enum: PascalCase.
|
||||
- 함수/변수/파라미터: camelCase.
|
||||
- 상수: UPPER_SNAKE_CASE (`companion object` 내부 `const val`).
|
||||
- Request/Response DTO는 `...Request`, `...Response` 접미사를 유지한다.
|
||||
- 서비스/컨트롤러/리포지토리 명명은 역할 접미사(`Service`, `Controller`, `Repository`)를 유지한다.
|
||||
|
||||
### 4) 타입/널 처리
|
||||
- Kotlin 타입 시스템을 활용하고 nullable(`?`)를 명시한다.
|
||||
- 불필요한 `Any`/약한 타입을 피하고 구체 타입을 우선한다.
|
||||
- 기존 코드에서 `!!` 사용이 많지만, 신규 코드는 가능한 안전 호출/가드절/명시적 예외로 대체를 우선 고려한다.
|
||||
|
||||
### 5) API/응답 규칙
|
||||
- API 응답은 `ApiResponse.ok(...)`, `ApiResponse.error(...)` 패턴을 따른다.
|
||||
- 컨트롤러는 도메인 예외를 직접 포맷하지 말고 `SodaException`을 던진다.
|
||||
- 인증 사용자 필요 시 `@AuthenticationPrincipal(... ) member: Member?` 패턴 + null 가드절을 사용한다.
|
||||
|
||||
### 6) 예외 처리 규칙
|
||||
- 비즈니스 예외는 `SodaException(messageKey = "...")` 우선 사용.
|
||||
- 사용자 노출 문구는 하드코딩보다 `messageKey` 기반 i18n을 우선한다.
|
||||
- 공통 예외 변환은 `SodaExceptionHandler`에서 수행하므로, 개별 컨트롤러에서 중복 처리하지 않는다.
|
||||
- 예외를 삼키는 빈 `catch` 블록을 금지한다.
|
||||
|
||||
### 7) 트랜잭션 규칙
|
||||
- 서비스 계층에서 `@Transactional`을 사용한다.
|
||||
- 조회 위주 메서드는 `@Transactional(readOnly = true)`를 우선한다.
|
||||
- 쓰기 로직은 메서드 단위 `@Transactional`로 경계를 명확히 한다.
|
||||
|
||||
### 8) 비동기/동시성 규칙
|
||||
- 비동기 처리는 Kotlin Coroutines 패턴을 따른다.
|
||||
- `CoroutineScope(Dispatchers.IO)` + `launch` + 예외 처리 패턴을 일관되게 유지한다.
|
||||
- 생명주기 종료 시 scope 정리(`@PreDestroy`) 패턴을 참고한다.
|
||||
|
||||
### 9) 의존성 주입
|
||||
- 생성자 주입(primary constructor + `private val`)을 기본으로 사용한다.
|
||||
- 필드 주입보다 명시적 생성자 주입을 우선한다.
|
||||
|
||||
### 10) 주석
|
||||
- 의미 단위별로 주석을 작성한다.
|
||||
- 주석은 한 문장으로 간결하게 작성한다.
|
||||
- 주석은 코드의 의도와 구조를 설명한다.
|
||||
- 주석은 코드 변경 시 업데이트를 잊지 않는다.
|
||||
|
||||
## 테스트 스타일 규칙
|
||||
- 테스트 프레임워크: JUnit 5 (`useJUnitPlatform()`)
|
||||
- 목킹: Mockito 사용 패턴 존재 (`Mockito.mock`, ``Mockito.`when`(...)``)
|
||||
- 검증: `assertEquals`, `assertThrows` 패턴 준수.
|
||||
- 테스트 이름은 의도가 드러나는 영어 문장형(`should...`)을 유지한다.
|
||||
- 테스트는 DisplayName으로 한국어 설명을 추가한다.
|
||||
- 예외 상황이 있는지 확인하고 예외 상황에 대한 테스트 케이스를 추가한다.
|
||||
|
||||
## 설정/보안 유의사항
|
||||
- `application.yml`은 다수의 `${ENV_VAR}`를 사용한다.
|
||||
- 비밀값(API Key, Secret, Token, DB 비밀번호)을 코드/문서/로그에 평문으로 남기지 않는다.
|
||||
- 환경변수/시크릿 파일은 커밋 대상에서 제외한다.
|
||||
|
||||
## Cursor/Copilot 규칙 반영
|
||||
`/.cursorrules`, `/.cursor/rules/`, `/.github/copilot-instructions.md` 파일은 현재 없다.
|
||||
별도 규칙 파일이 추가되면 본 문서보다 해당 규칙을 우선 반영한다.
|
||||
|
||||
## 커밋 메시지 규칙 (표준 Conventional Commits)
|
||||
## 커밋 메시지 규칙
|
||||
- 커밋 상세 가이드/절차는 `.opencode/skills/commit-policy/SKILL.md`를 단일 기준으로 사용한다.
|
||||
- 커밋 작업 시작 시 `skill` 도구로 `commit-policy`를 먼저 로드한다.
|
||||
- 기본 형식은 `<type>(scope): <description>`를 사용한다.
|
||||
- `type`은 소문자(`feat`, `fix`, `chore`, `docs`, `refactor`, `test` 등)를 사용한다.
|
||||
- 제목(description)은 한글로 작성하고, 명령형/간결한 현재형으로 작성한다.
|
||||
- 이슈 참조 footer는 `Refs: #123` 또는 `Refs: #123, #456` 형식을 사용한다.
|
||||
- 커밋 본문에는 `Ultraworked with [Sisyphus]...` 및 `Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>` 자동 footer를 포함하지 않는다.
|
||||
- `git commit` 실행 직전과 직후에 `work/scripts/check-commit-message-rules.sh`를 실행해 규칙 준수 여부를 검증한다.
|
||||
|
||||
### 커밋 메시지 검증 절차
|
||||
- `git commit` 실행 직전에 `work/scripts/check-commit-message-rules.sh`를 실행해 규칙 준수 여부를 확인한다.
|
||||
- `git commit` 실행 직후에도 `work/scripts/check-commit-message-rules.sh`를 다시 실행해 최종 메시지를 재검증한다.
|
||||
- 스크립트 결과가 `[FAIL]`이면 커밋 메시지를 규칙에 맞게 수정한 뒤 다시 검증한다.
|
||||
|
||||
## 작업 절차 체크리스트
|
||||
- 변경 전: 유사 기능 코드를 먼저 찾아 네이밍/예외/응답 패턴을 맞춘다.
|
||||
- 변경 중: 공개 API 스키마를 임의 변경하지 말고, 작은 단위로 안전하게 수정한다.
|
||||
- 변경 후: 최소 단일 테스트 또는 `./gradlew test`를 실행하고, 필요 시 `./gradlew ktlintCheck`를 수행한다.
|
||||
- 커밋 전/후: `commit-policy` 스킬을 먼저 로드하고, `git commit` 직전과 직후에 `work/scripts/check-commit-message-rules.sh`를 실행해 커밋 메시지 규칙 준수 여부를 확인한다.
|
||||
|
||||
## 작업 계획 문서 규칙 (docs)
|
||||
- 모든 작업 시작 전에 `docs` 폴더 아래에 계획 문서를 먼저 생성하고, 해당 문서를 기준으로 구현을 진행한다.
|
||||
- 계획 문서 파일명은 `[날짜]_구현할내용한글.md` 형식을 사용한다.
|
||||
## PRD 및 계획 TASK 문서 규칙 (docs)
|
||||
- PRD와 계획 TASK 문서 없이 구현하지 않는다.
|
||||
- 작업 문서 작성과 구현은 반드시 `사용자 프롬프트 입력 -> PRD 문서 작성 -> 모호한 사항 사용자 인터뷰 -> 인터뷰 내용으로 PRD 보강 -> PRD 기반 계획 TASK 문서 작성 -> 계획 TASK 기반 최소 구현` 순서로 진행한다.
|
||||
- PRD 작성 중 애매하거나 더 필요한 내용, 결정해야 하는 사항이 있으면 애매한 사항이 없어질 때까지 사용자와 인터뷰한다.
|
||||
- PRD 문서는 `docs/prd/` 아래에 작성하고, `docs/prd/sample-prd.md`에서 필요한 섹션만 발췌해 작성한다.
|
||||
- 계획 TASK 문서는 `docs/plan-task/` 아래에 작성하고, 해당 문서를 기준으로 구현을 진행한다.
|
||||
- 연속된 하나의 작업에 대한 후속 수정/보완이라면 새 PRD 또는 계획 TASK 문서를 만들지 말고 기존 문서에 요구사항, 작업 항목, 검증 기록을 이어서 추가한다.
|
||||
- PRD 문서 파일명은 `[날짜]_구현할내용한글_prd.md` 형식을 사용해 계획 TASK 문서와 구분한다.
|
||||
- 계획 TASK 문서 파일명은 `[날짜]_구현할내용한글.md` 형식을 사용한다.
|
||||
- 날짜는 `YYYYMMDD` 8자리 숫자를 사용한다.
|
||||
- 파일명 예시: `20260101_구글계정으로로그인.md`
|
||||
- 구현 항목은 기능/작업 단위로 분리해 체크박스(`- [ ]`) 목록으로 작성한다.
|
||||
- 구현 완료 시마다 체크박스를 `- [x]`로 갱신하고, 각 항목이 정상 구현되었는지 확인한다.
|
||||
- 작업 도중 범위가 변경되면 계획 문서의 체크박스 항목을 먼저 업데이트한 뒤 구현을 진행한다.
|
||||
- 모든 구현이 끝난 후 결과 보고 시 계획 문서 맨 아래에 무엇을, 왜, 어떻게 검증했는지 한국어로 간단히 기록한다.
|
||||
- 후속 수정이 발생해도 기존 검증 기록은 삭제/덮어쓰지 않고 누적한다(예: `1차 구현`, `2차 수정`).
|
||||
- 검증 기록은 단계별로 `무엇을/왜/어떻게`를 유지해 작성하고, 이전 단계와 구분이 되도록 명시한다.
|
||||
- 단계별 `어떻게`에는 실제 실행한 검증 명령과 결과(성공/실패/불가 사유)를 함께 기록한다.
|
||||
- 기존 기록 정정이 필요하면 원문을 지우지 말고 `정정` 항목을 추가해 사유와 변경 내용을 남긴다.
|
||||
|
||||
## 문서 유지보수 규칙
|
||||
- `build.gradle.kts` 변경 시 실행 명령 섹션을 함께 갱신한다.
|
||||
- 테스트 클래스 추가/이동 시 단일 테스트 실행 예시를 최신 상태로 유지한다.
|
||||
- `.editorconfig` 변경 시 포맷 규칙 섹션을 동기화한다.
|
||||
- Cursor/Copilot 규칙 파일이 생기면 해당 내용을 이 문서에 반영한다.
|
||||
- 문서 변경 후 최소 한 번 `./gradlew tasks --all`로 명령 유효성을 확인한다.
|
||||
- 불확실한 규칙은 추측으로 채우지 말고 근거 파일 경로를 먼저 확인한다.
|
||||
- 에이전트 안내 문구는 한국어 중심으로 유지한다.
|
||||
- 커밋 규칙 예시는 팀 컨벤션 변경 시 즉시 업데이트한다.
|
||||
- 계획 TASK 문서의 구현 항목은 기능/작업 단위로 분리해 체크박스(`- [ ]`) 목록으로 작성한다.
|
||||
- 구현 완료 시마다 체크박스를 `- [x]`로 갱신하고, 범위가 바뀌면 문서를 먼저 갱신한다.
|
||||
- 결과 보고 시 계획 TASK 문서 맨 아래에 무엇을, 왜, 어떻게 검증했는지 한국어로 누적 기록한다.
|
||||
|
||||
## 에이전트 동작 원칙
|
||||
- 추측하지 말고, 근거 파일을 읽고 결정한다.
|
||||
|
||||
15
docs/agent-guides/문서유지보수.md
Normal file
15
docs/agent-guides/문서유지보수.md
Normal file
@@ -0,0 +1,15 @@
|
||||
# 문서 유지보수
|
||||
|
||||
## 문서 유지보수 규칙
|
||||
- PRD 문서는 `docs/prd/`에 두고, 계획 TASK 문서는 `docs/plan-task/`에 둔다.
|
||||
- PRD 문서 파일명은 `[날짜]_구현할내용한글_prd.md`, 계획 TASK 문서 파일명은 `[날짜]_구현할내용한글.md` 형식을 사용한다.
|
||||
- PRD 문서는 `docs/prd/sample-prd.md`에서 필요한 섹션만 발췌해 작성하고, 불필요한 빈 섹션을 기계적으로 복사하지 않는다.
|
||||
- `build.gradle.kts` 변경 시 실행 명령 섹션을 함께 갱신한다.
|
||||
- 테스트 클래스 추가/이동 시 단일 테스트 실행 예시를 최신 상태로 유지한다.
|
||||
- `.editorconfig` 변경 시 포맷 규칙 섹션을 동기화한다.
|
||||
- Cursor/Copilot 규칙 파일이 생기면 해당 내용을 이 문서에 반영한다.
|
||||
- 연속된 하나의 작업에 대해 PRD 또는 계획 TASK 문서가 여러 개 생기지 않도록 기존 문서 재사용 여부를 먼저 확인한다.
|
||||
- 문서 변경 후 최소 한 번 `./gradlew tasks --all`로 명령 유효성을 확인한다.
|
||||
- 불확실한 규칙은 추측으로 채우지 말고 근거 파일 경로를 먼저 확인한다.
|
||||
- 에이전트 안내 문구는 한국어 중심으로 유지한다.
|
||||
- 커밋 규칙 예시는 팀 컨벤션 변경 시 즉시 업데이트한다.
|
||||
6
docs/agent-guides/설정보안.md
Normal file
6
docs/agent-guides/설정보안.md
Normal file
@@ -0,0 +1,6 @@
|
||||
# 설정 보안
|
||||
|
||||
## 설정/보안 유의사항
|
||||
- `application.yml`은 다수의 `${ENV_VAR}`를 사용한다.
|
||||
- 비밀값(API Key, Secret, Token, DB 비밀번호)을 코드/문서/로그에 평문으로 남기지 않는다.
|
||||
- 환경변수/시크릿 파일은 커밋 대상에서 제외한다.
|
||||
13
docs/agent-guides/작업절차.md
Normal file
13
docs/agent-guides/작업절차.md
Normal file
@@ -0,0 +1,13 @@
|
||||
# 작업 절차
|
||||
|
||||
## 작업 절차 체크리스트
|
||||
- 변경 전: PRD와 계획 TASK 문서 없이 구현하지 않는다.
|
||||
- 변경 전: 사용자 프롬프트를 받으면 먼저 `docs/prd/` 아래에 PRD 문서를 작성하고, `docs/prd/sample-prd.md`에서 필요한 섹션만 발췌한다.
|
||||
- 변경 전: PRD 작성 중 애매하거나 더 필요한 내용, 결정해야 하는 사항이 있으면 애매한 사항이 없어질 때까지 사용자와 인터뷰하고 PRD를 보강한다.
|
||||
- 변경 전: 보강된 PRD를 바탕으로 `docs/plan-task/` 아래에 계획 TASK 문서를 작성한 뒤, 해당 문서를 기준으로 필요한 내용만 최소 구현한다.
|
||||
- 변경 전: 유사 기능 코드를 먼저 찾아 네이밍/예외/응답 패턴을 맞춘다.
|
||||
- 변경 전: 같은 작업의 연속 후속 수정인지 먼저 확인하고, 연속 작업이면 새 PRD 또는 계획 TASK 문서를 만들지 말고 기존 문서를 갱신한다.
|
||||
- 변경 중: 공개 API 스키마를 임의 변경하지 말고, 작은 단위로 안전하게 수정한다.
|
||||
- 변경 후: 최소 단일 테스트 또는 `./gradlew test`를 실행하고, 필요 시 `./gradlew ktlintCheck`를 수행한다.
|
||||
- 커밋 전/후: `commit-policy` 스킬을 먼저 로드하고, `git commit` 직전과 직후에 `work/scripts/check-commit-message-rules.sh`를 실행해 커밋 메시지 규칙 준수 여부를 확인한다.
|
||||
- 커밋 전/후 확인 시 Sisyphus attribution footer가 없는지 함께 검증한다.
|
||||
63
docs/agent-guides/코드스타일.md
Normal file
63
docs/agent-guides/코드스타일.md
Normal file
@@ -0,0 +1,63 @@
|
||||
# 코드 스타일
|
||||
|
||||
## 코드 스타일 규칙
|
||||
|
||||
### 1) 포맷/기본 규칙
|
||||
- `.editorconfig` 기준을 준수한다.
|
||||
- 인덴트: 공백 4칸.
|
||||
- 줄바꿈: LF.
|
||||
- 최대 라인 길이: 130.
|
||||
- 파일 끝 개행 유지, trailing whitespace 제거.
|
||||
|
||||
### 2) import 규칙
|
||||
- 와일드카드 import(`*`)를 사용하지 않는다.
|
||||
- 사용하지 않는 import를 남기지 않는다.
|
||||
- import alias(`as`)는 현재 코드베이스에서 사용 사례가 없으므로 지양한다.
|
||||
- 기존 파일의 import 정렬/그룹 스타일을 그대로 맞춘다.
|
||||
|
||||
### 3) 네이밍 규칙
|
||||
- 클래스/인터페이스/enum: PascalCase.
|
||||
- 함수/변수/파라미터: camelCase.
|
||||
- 상수: UPPER_SNAKE_CASE (`companion object` 내부 `const val`).
|
||||
- Request/Response DTO는 `...Request`, `...Response` 접미사를 유지한다.
|
||||
- 서비스/컨트롤러/리포지토리 명명은 역할 접미사(`Service`, `Controller`, `Repository`)를 유지한다.
|
||||
|
||||
### 4) 패키지/코드 배치 규칙
|
||||
- 기존 로직을 수정하는 경우에는 기존 패키지 구조를 따른다.
|
||||
- 기존 로직 수정이 아닌 신규 API나 하위 코드는 `kr.co.vividnext.sodalive.v2` 패키지 하위에 작성한다.
|
||||
|
||||
### 5) 타입/널 처리
|
||||
- Kotlin 타입 시스템을 활용하고 nullable(`?`)를 명시한다.
|
||||
- 불필요한 `Any`/약한 타입을 피하고 구체 타입을 우선한다.
|
||||
- 기존 코드에서 `!!` 사용이 많지만, 신규 코드는 가능한 안전 호출/가드절/명시적 예외로 대체를 우선 고려한다.
|
||||
|
||||
### 6) API/응답 규칙
|
||||
- API 응답은 `ApiResponse.ok(...)`, `ApiResponse.error(...)` 패턴을 따른다.
|
||||
- 컨트롤러는 도메인 예외를 직접 포맷하지 말고 `SodaException`을 던진다.
|
||||
- 인증 사용자 필요 시 `@AuthenticationPrincipal(... ) member: Member?` 패턴 + null 가드절을 사용한다.
|
||||
|
||||
### 7) 예외 처리 규칙
|
||||
- 비즈니스 예외는 `SodaException(messageKey = "...")` 우선 사용.
|
||||
- 사용자 노출 문구는 하드코딩보다 `messageKey` 기반 i18n을 우선한다.
|
||||
- 공통 예외 변환은 `SodaExceptionHandler`에서 수행하므로, 개별 컨트롤러에서 중복 처리하지 않는다.
|
||||
- 예외를 삼키는 빈 `catch` 블록을 금지한다.
|
||||
|
||||
### 8) 트랜잭션 규칙
|
||||
- 서비스 계층에서 `@Transactional`을 사용한다.
|
||||
- 조회 위주 메서드는 `@Transactional(readOnly = true)`를 우선한다.
|
||||
- 쓰기 로직은 메서드 단위 `@Transactional`로 경계를 명확히 한다.
|
||||
|
||||
### 9) 비동기/동시성 규칙
|
||||
- 비동기 처리는 Kotlin Coroutines 패턴을 따른다.
|
||||
- `CoroutineScope(Dispatchers.IO)` + `launch` + 예외 처리 패턴을 일관되게 유지한다.
|
||||
- 생명주기 종료 시 scope 정리(`@PreDestroy`) 패턴을 참고한다.
|
||||
|
||||
### 10) 의존성 주입
|
||||
- 생성자 주입(primary constructor + `private val`)을 기본으로 사용한다.
|
||||
- 필드 주입보다 명시적 생성자 주입을 우선한다.
|
||||
|
||||
### 11) 주석
|
||||
- 의미 단위별로 주석을 작성한다.
|
||||
- 주석은 한 문장으로 간결하게 작성한다.
|
||||
- 주석은 코드의 의도와 구조를 설명한다.
|
||||
- 주석은 코드 변경 시 업데이트를 잊지 않는다.
|
||||
9
docs/agent-guides/테스트스타일.md
Normal file
9
docs/agent-guides/테스트스타일.md
Normal file
@@ -0,0 +1,9 @@
|
||||
# 테스트 스타일
|
||||
|
||||
## 테스트 스타일 규칙
|
||||
- 테스트 프레임워크: JUnit 5 (`useJUnitPlatform()`)
|
||||
- 목킹: Mockito 사용 패턴 존재 (`Mockito.mock`, ``Mockito.`when`(...)``)
|
||||
- 검증: `assertEquals`, `assertThrows` 패턴 준수.
|
||||
- 테스트 이름은 의도가 드러나는 영어 문장형(`should...`)을 유지한다.
|
||||
- 테스트는 DisplayName으로 한국어 설명을 추가한다.
|
||||
- 예외 상황이 있는지 확인하고 예외 상황에 대한 테스트 케이스를 추가한다.
|
||||
10
docs/plan-task/20260406_omxgitignore.md
Normal file
10
docs/plan-task/20260406_omxgitignore.md
Normal file
@@ -0,0 +1,10 @@
|
||||
# `.omx/` Git 제외 처리
|
||||
|
||||
- [x] `.gitignore`에 `.omx/`를 추가한다.
|
||||
- [x] `git status`와 `git check-ignore`로 제외가 정상 동작하는지 확인한다.
|
||||
|
||||
## 검증 기록
|
||||
|
||||
- 무엇을: `.gitignore`에 `.omx/`를 추가하고 `.omx/` 하위 파일들이 무시되는지 확인했다.
|
||||
- 왜: `.omx/`는 런타임 상태, 로그, 메트릭 파일이라 버전 관리 대상이 아니기 때문이다.
|
||||
- 어떻게: `git check-ignore -v .omx/tmux-hook.json .omx/state/hud-state.json .omx/logs/turns-2026-04-06.jsonl`로 무시 규칙을 확인했고, `git status --short`로 `.omx/`가 더 이상 추적 대상이 아니고 문서만 남는지 확인했다.
|
||||
@@ -0,0 +1,19 @@
|
||||
SET @schema_name := DATABASE();
|
||||
|
||||
SET @settlement_ratio_column_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.columns
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'content'
|
||||
AND column_name = 'settlement_ratio'
|
||||
);
|
||||
|
||||
SET @add_settlement_ratio_column_sql := IF(
|
||||
@settlement_ratio_column_exists = 0,
|
||||
'ALTER TABLE content ADD COLUMN settlement_ratio INT NULL COMMENT ''콘텐츠별 정산 요율(%)'' AFTER price',
|
||||
'SELECT ''content.settlement_ratio already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_settlement_ratio_column_stmt FROM @add_settlement_ratio_column_sql;
|
||||
EXECUTE add_settlement_ratio_column_stmt;
|
||||
DEALLOCATE PREPARE add_settlement_ratio_column_stmt;
|
||||
22
docs/plan-task/20260407_커밋footer자동추가차단.md
Normal file
22
docs/plan-task/20260407_커밋footer자동추가차단.md
Normal file
@@ -0,0 +1,22 @@
|
||||
# 20260407 커밋 footer 자동 추가 차단
|
||||
|
||||
## 구현 계획
|
||||
- [x] oh-my-openagent 기본 footer 동작과 저장소 로컬 커밋 워크플로우의 영향 범위를 문서화한다.
|
||||
- [x] `AGENTS.md`에 커밋 본문에서 Sisyphus footer와 자동 `Co-authored-by` 라인을 허용하지 않는 규칙을 추가한다.
|
||||
- [x] `.opencode/skills/commit-policy/SKILL.md`에 검증된 메시지를 그대로 `git commit`에 전달하고 자동 footer를 금지하는 절차를 반영한다.
|
||||
- [x] `.opencode/commands/commit.md`에 `/commit` 커맨드가 자동 footer 없는 최종 메시지를 사용하도록 지시를 보강한다.
|
||||
- [x] `work/scripts/check-commit-message-rules.sh`에 Sisyphus footer 및 자동 `Co-authored-by` 라인 차단 검증을 추가한다.
|
||||
- [x] 변경 문서와 스크립트에 대해 진단 및 실행 검증을 수행한다.
|
||||
|
||||
## 검증 기록
|
||||
- [x] 작업 완료 후 검증 결과를 기록한다.
|
||||
|
||||
- 1차 구현
|
||||
- 무엇을: `AGENTS.md`, `.opencode/skills/commit-policy/SKILL.md`, `.opencode/commands/commit.md`, `work/scripts/check-commit-message-rules.sh`를 수정해 커밋 본문에서 `Ultraworked with [Sisyphus]...`와 `Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>` 자동 footer를 금지하고, `/commit` 경로가 검증된 메시지를 그대로 `git commit`에 전달하도록 명시했다.
|
||||
- 왜: oh-my-openagent 기본 설정과 알려진 버그로 자동 footer가 붙을 수 있으므로, 저장소 로컬 규칙과 검증 스크립트에서 이를 명시적으로 차단해야 커밋 결과를 일관되게 통제할 수 있기 때문이다.
|
||||
- 어떻게: `lsp_diagnostics`로 `AGENTS.md`, `.opencode/skills/commit-policy/SKILL.md`, `.opencode/commands/commit.md`, `work/scripts/check-commit-message-rules.sh`, `docs/20260407_커밋footer자동추가차단.md`에 대해 모두 `No diagnostics found`를 확인했다. `bash -n work/scripts/check-commit-message-rules.sh`로 문법을 검증했고, `./work/scripts/check-commit-message-rules.sh --message`로 정상 메시지/`Refs` footer 허용 케이스는 PASS, Sisyphus footer와 자동 `Co-authored-by` 케이스는 FAIL을 확인했다. 추가로 `./gradlew tasks --all` 실행 결과 `BUILD SUCCESSFUL`을 확인했다.
|
||||
|
||||
- 2차 수정
|
||||
- 무엇을: Oracle 검토 의견을 반영해 `.opencode/skills/commit-policy/SKILL.md`와 `.opencode/commands/commit.md`에서 `--message-file` 검증 후 같은 파일을 `git commit -F`에 전달하는 경로를 권장하도록 보강했고, `work/scripts/check-commit-message-rules.sh`의 `Co-authored-by` 차단 조건을 공백 변형까지 탐지하도록 확장했다.
|
||||
- 왜: exact string 하나만 금지하면 footer 형식이 조금만 달라져도 놓칠 수 있으므로, 외부 기본 동작이나 버그로 인한 변형까지 더 안정적으로 차단해야 하기 때문이다.
|
||||
- 어떻게: `lsp_diagnostics`로 `.opencode/skills/commit-policy/SKILL.md`, `.opencode/commands/commit.md`, `work/scripts/check-commit-message-rules.sh`에 대해 모두 `No diagnostics found`를 확인했다. `bash -n work/scripts/check-commit-message-rules.sh`를 다시 실행해 문법을 검증했고, `./work/scripts/check-commit-message-rules.sh --message`로 기본 메시지와 `Refs` footer는 PASS, Sisyphus footer/기본 `Co-authored-by`/공백 변형 `Co-authored-by` 케이스는 모두 FAIL을 확인했다.
|
||||
85
docs/plan-task/20260407_콘텐츠별정산요율추가.md
Normal file
85
docs/plan-task/20260407_콘텐츠별정산요율추가.md
Normal file
@@ -0,0 +1,85 @@
|
||||
- [x] 변수명 확정: 엔티티 내부 추가 변수는 `AudioContent.settlementRatio: Int?`로 사용한다.
|
||||
- 이유: `AudioContent`는 이미 콘텐츠 도메인 엔티티이므로 `contentSettlementRatio`는 중복 표현에 가깝다.
|
||||
- 근거: 이 저장소의 엔티티 필드는 `AudioContent.price`, `LiveRoom.price`, `CreatorCommunity.price`처럼 엔티티 스코프 안에서는 도메인 접두어를 반복하지 않는다.
|
||||
- 예외 기준: `CreatorSettlementRatio.contentSettlementRatio`처럼 하나의 엔티티 안에서 `live/content/community` 여러 정산 대상을 함께 구분해야 할 때만 `content` 접두어가 필요하다.
|
||||
- DTO/API 정책: 해당 값은 관리자에서만 사용하므로 관리자 요청/응답 DTO와 API 필드명도 예외 없이 `settlementRatio`로 통일한다.
|
||||
- nullable 정책: 기존 데이터와 크리에이터 정산 요율 미등록 케이스를 안전하게 수용하기 위해 초기 도입 시 `Int?`로 두고, 계산 시 `콘텐츠별 요율 -> 크리에이터 기본 요율 -> 70% 기본값` 순서로 fallback 하도록 설계한다.
|
||||
|
||||
- [x] `AudioContent` 엔티티에 콘텐츠별 정산 요율 필드를 추가한다.
|
||||
- 대상 파일: `src/main/kotlin/kr/co/vividnext/sodalive/content/AudioContent.kt`
|
||||
- 작업 내용: `price` 인접 위치에 `settlementRatio: Int?` 필드를 추가하고, 기존 생성자 호출부가 모두 컴파일되도록 생성 경로를 함께 정리한다.
|
||||
|
||||
- [x] 관리자 콘텐츠 목록 조회 응답에 콘텐츠별 정산 요율을 노출한다.
|
||||
- 대상 파일: `src/main/kotlin/kr/co/vividnext/sodalive/admin/content/AdminContentRepository.kt`, `src/main/kotlin/kr/co/vividnext/sodalive/admin/content/GetAdminContentListResponse.kt`, `src/main/kotlin/kr/co/vividnext/sodalive/admin/content/AdminContentService.kt`
|
||||
- 작업 내용:
|
||||
- `QGetAdminContentListItem(...)` QueryProjection에 `audioContent.settlementRatio`를 추가한다.
|
||||
- `GetAdminContentListItem`에 `settlementRatio: Int?`를 추가하고, 관리자 목록 응답 필드명도 동일하게 `settlementRatio`로 맞춘다.
|
||||
- `AdminContentController.getAudioContentList` 응답에 정산 요율이 함께 내려가도록 조회 체인을 맞춘다.
|
||||
|
||||
- [x] 관리자 콘텐츠 수정 API에서 콘텐츠별 정산 요율을 수정할 수 있게 한다.
|
||||
- 대상 파일: `src/main/kotlin/kr/co/vividnext/sodalive/admin/content/UpdateAdminContentRequest.kt`, `src/main/kotlin/kr/co/vividnext/sodalive/admin/content/AdminContentService.kt`
|
||||
- 작업 내용:
|
||||
- `UpdateAdminContentRequest`에 `settlementRatio: Int?`를 추가하고, 관리자 수정 요청 필드명도 동일하게 `settlementRatio`로 맞춘다.
|
||||
- `AdminContentService.updateAudioContent`에서 요청값이 들어오면 `audioContent.settlementRatio`를 갱신한다.
|
||||
- 숫자 범위 정책은 `0~100`으로 검증한다.
|
||||
- 개별 콘텐츠 정산 요율 삭제는 `isSettlementRatioDeleted: true` 플래그로만 처리하고, `settlementRatio`와 동시 전달 시 invalid request 로 처리한다.
|
||||
|
||||
- [x] 실제 콘텐츠 정산 계산이 크리에이터 기본 요율이 아니라 콘텐츠별 요율을 우선 사용하도록 변경한다.
|
||||
- 대상 파일: `src/main/kotlin/kr/co/vividnext/sodalive/admin/calculate/AdminCalculateQueryRepository.kt`, `src/main/kotlin/kr/co/vividnext/sodalive/creator/admin/calculate/CreatorAdminCalculateQueryRepository.kt`, `src/main/kotlin/kr/co/vividnext/sodalive/admin/calculate/GetCalculateContentQueryData.kt`
|
||||
- 작업 내용:
|
||||
- 콘텐츠 판매/누적 판매 집계 쿼리에서 `creatorSettlementRatio.contentSettlementRatio` 직접 사용 부분을 점검한다.
|
||||
- 정산 대상 비율은 `audioContent.settlementRatio`를 우선 사용하고, 값이 없을 때만 `creatorSettlementRatio.contentSettlementRatio`를 fallback 하도록 쿼리 또는 계산 DTO를 조정한다.
|
||||
- 현재 `GetCalculateContentQueryData`의 70% 기본값 정책은 마지막 fallback 으로 유지한다.
|
||||
|
||||
- [x] 크리에이터 관리자 정산 조회도 동일 기준을 사용하도록 맞춘다.
|
||||
- 대상 파일: `src/main/kotlin/kr/co/vividnext/sodalive/creator/admin/calculate/CreatorAdminCalculateQueryRepository.kt`
|
||||
- 작업 내용: 관리자 정산 조회와 동일하게 콘텐츠별 정산 요율 우선 정책을 반영해 관리자/크리에이터 화면 간 계산 기준이 달라지지 않게 한다.
|
||||
|
||||
- [x] 기존 데이터 처리 정책을 정리한다.
|
||||
- 대상 범위: 운영 DB 스키마/기존 콘텐츠 데이터
|
||||
- 작업 내용:
|
||||
- 신규 컬럼 추가 시 nullable 로 도입하고, DDL은 `docs/20260407_audio_content_settlement_ratio_ddl.sql` 기준으로 관리한다.
|
||||
- 콘텐츠 등록 시 크리에이터 기본 정산 요율을 복사하지 않고, `NULL` 상태에서도 계산이 가능하도록 fallback 순서를 유지한다.
|
||||
- 운영 정책상 기존 콘텐츠에도 즉시 고정값이 필요하면 별도 SQL 또는 배치 백필 계획을 추가로 작성한다.
|
||||
|
||||
- [x] 영향 DTO/Q 클래스/컴파일 산출물을 재생성하고 검증한다.
|
||||
- 작업 내용:
|
||||
- QueryDSL projection 변경 후 Q 클래스 재생성이 필요한지 확인하고 빌드로 반영한다.
|
||||
- 엔티티/관리자 DTO/API/QueryProjection 필드명이 모두 `settlementRatio`로 일치하는지 확인해 매핑 누락 가능성을 제거한다.
|
||||
|
||||
- [x] 검증을 단계별로 수행한다.
|
||||
- 작업 내용:
|
||||
- `AdminContentController.getAudioContentList`에서 정산 요율 조회 포함 여부를 확인한다.
|
||||
- `AdminContentController.modifyAudioContent`에서 정산 요율 수정 반영 여부를 확인한다.
|
||||
- `AdminContentController.modifyAudioContent`에서 `isSettlementRatioDeleted = true` 요청 시 개별 콘텐츠 정산 요율이 삭제되는지 확인한다.
|
||||
- 콘텐츠 등록 시 `settlementRatio`가 nullable 상태로 유지되어도 계산 fallback 이 정상 동작하는지 확인한다.
|
||||
- 콘텐츠 정산 조회(`AdminCalculateQueryRepository`, `CreatorAdminCalculateQueryRepository`)가 콘텐츠별 요율을 우선 적용하는지 확인한다.
|
||||
- 실행 검증은 최소 `./gradlew build`, 필요 시 `./gradlew test`까지 수행한다.
|
||||
|
||||
## 1차 구현 검증 기록
|
||||
|
||||
- 무엇을: `AudioContent.settlementRatio` nullable 컬럼 추가, 관리자 목록/수정 API 반영, 콘텐츠/누적 정산 쿼리의 콘텐츠별 요율 우선 fallback 반영, 생성 시 기본 요율 복사 제거.
|
||||
- 왜: 기존 콘텐츠와 미설정 콘텐츠를 `NULL`로 유지하면서도 관리자에서 개별 요율을 조회/수정하고 정산 시 올바른 fallback 순서를 적용하기 위해.
|
||||
- 어떻게:
|
||||
- `./gradlew build` → 성공
|
||||
- `./gradlew test` → `build` 과정에 포함되어 성공
|
||||
- 관리자/정산 API의 실서버 수동 호출 검증 → 이 로컬 작업 세션에서는 애플리케이션 실행 및 인증 가능한 테스트 데이터가 없어 미실행
|
||||
|
||||
## 2차 수정 검증 기록
|
||||
|
||||
- 무엇을: `@SpringBootTest` 없이 콘텐츠 정산 계산 DTO의 명시 비율 적용과 `null -> 70% fallback`을 검증하는 순수 단위 테스트를 추가했다.
|
||||
- 왜: 정산 쿼리 변경만으로는 계산 결과가 코드상에서 충분히 고정되지 않아, 문서에 적힌 계산 규칙을 재현 가능한 테스트로 보장하기 위해.
|
||||
- 어떻게:
|
||||
- `./gradlew test --tests kr.co.vividnext.sodalive.admin.calculate.ContentSettlementCalculationTest` → 성공
|
||||
- 검증 대상: `GetCalculateContentQueryData`, `GetCumulativeSalesByContentQueryData`
|
||||
- 검증 시나리오: `settlementRatio = 80` 적용, `settlementRatio = null` 시 70% fallback 적용
|
||||
|
||||
## 3차 수정 검증 기록
|
||||
|
||||
- 무엇을: 관리자 콘텐츠 수정 API의 개별 콘텐츠 정산 요율 삭제 방식을 명시적 null 대신 `isSettlementRatioDeleted` 플래그로 전환한다.
|
||||
- 왜: 부분 업데이트 요청에서 필드 생략과 null 삭제 의미가 섞이지 않도록 API 계약을 명확히 하기 위해.
|
||||
- 어떻게:
|
||||
- `./gradlew test --tests kr.co.vividnext.sodalive.admin.content.AdminContentServiceTest` → 성공
|
||||
- `./gradlew test --tests kr.co.vividnext.sodalive.admin.calculate.ContentSettlementCalculationTest` → 성공
|
||||
- `./gradlew build` → 성공
|
||||
- 검증 시나리오: 유효한 요율 설정, 삭제 플래그 삭제, 값/삭제 플래그 동시 전달 충돌, `null` 키/삭제 플래그 동시 전달 충돌, 범위 초과 거부, 정산 fallback 회귀 확인
|
||||
658
docs/plan-task/20260408_에이전트권한및정산기능추가.md
Normal file
658
docs/plan-task/20260408_에이전트권한및정산기능추가.md
Normal file
@@ -0,0 +1,658 @@
|
||||
# 에이전트 권한 및 정산 기능 추가 작업 계획
|
||||
|
||||
## 요구사항 상세 분석
|
||||
|
||||
### 1. 권한/역할 분석
|
||||
- `MemberRole.AGENT`는 이미 `Member.kt`에 정의되어 있으므로 역할 enum 자체의 신규 추가는 필요하지 않다.
|
||||
- 다만 현재 코드베이스에는 에이전트 전용 소속 관리/정산 조회 모듈이 없으므로, 실제 기능은 신규 구현이 필요하다.
|
||||
- 크리에이터를 에이전트에 소속하거나 해제하는 작업은 `ADMIN`만 수행할 수 있어야 한다.
|
||||
- 에이전트 전용 조회 API는 `AGENT` 권한 계정만 자신의 소속 크리에이터 데이터에 한해 접근할 수 있어야 한다.
|
||||
- 관리자용 권한 경계는 기존 `@PreAuthorize("hasRole('ADMIN')")`, 에이전트용 권한 경계는 `@PreAuthorize("hasRole('AGENT')")` 패턴을 따른다.
|
||||
|
||||
### 2. 관계 모델 분석
|
||||
- 요구사항은 `에이전트 1 : N 크리에이터`, `크리에이터 1 : 0..1 에이전트` 관계다.
|
||||
- 이 관계는 `Member` 자체에 필드를 직접 늘리는 방식보다, `kr.co.vividnext.sodalive.partner.agent` 패키지 안에 전용 연관 엔티티를 두는 편이 패키지 경계와 제약 관리에 더 적합하다.
|
||||
- 신규 연관 엔티티는 `creator_id` 유니크 제약으로 "한 크리에이터는 하나의 에이전트에만 소속"을 강제해야 한다.
|
||||
- 서비스 계층에서는 다음을 모두 검증해야 한다.
|
||||
- agent 회원이 실제 `MemberRole.AGENT`인지
|
||||
- creator 회원이 실제 `MemberRole.CREATOR`인지
|
||||
- creator가 이미 다른 agent에 소속되어 있는지
|
||||
- 자기 자신을 agent/creator로 잘못 연결하는 요청이 아닌지
|
||||
|
||||
### 3. 정산 규칙 분석
|
||||
- 에이전트 정산 비율은 관리자 설정값 1개만 있으면 된다.
|
||||
- 기준 금액은 "크리에이터 세전 정산금액(settlementAmount)"이며, 라이브/콘텐츠/커뮤니티처럼 항목별 비율이 아니라 최종 정산금액에 대해 에이전트 비율을 적용한다.
|
||||
- 에이전트 정산금은 크리에이터 정산금에서 차감하지 않고 별도로 계산한다.
|
||||
- 에이전트 정산금 계산식은 다음으로 고정한다.
|
||||
- `agentSettlementAmount = round(creatorSettlementAmount * agentSettlementRatio / 100)`
|
||||
- 크리에이터 입금액 계산식은 기존 로직을 그대로 유지한다.
|
||||
- 라이브/콘텐츠/커뮤니티는 기존 `CreatorSettlementRatio` 또는 콘텐츠별 정산 비율을 이용해 `settlementAmount`를 계산한 뒤, 그 결과에 에이전트 비율을 곱해야 한다.
|
||||
- 채널후원/콘텐츠후원도 기존 정산 계산 결과의 `settlementAmount`를 기준으로 에이전트 금액을 별도 계산해야 한다.
|
||||
|
||||
### 4. 조회 요구사항 분석
|
||||
- 에이전트는 소속 크리에이터 목록을 조회할 수 있어야 한다.
|
||||
- 에이전트는 소속 크리에이터 기준으로 아래 5개 현황을 조회할 수 있어야 한다.
|
||||
- 라이브
|
||||
- 콘텐츠 판매
|
||||
- 커뮤니티
|
||||
- 채널후원
|
||||
- 콘텐츠후원
|
||||
- 각 조회는 `/admin/calculate/content-by-creator` 계열과 유사하게 크리에이터별 집계 응답을 제공해야 한다.
|
||||
- 각 응답은 최소한 다음 정보를 포함해야 한다.
|
||||
- 크리에이터 식별 정보
|
||||
- 건수
|
||||
- 총 캔 수
|
||||
- 원화
|
||||
- 수수료
|
||||
- 정산금액
|
||||
- 합계(total)
|
||||
- 에이전트 정산금액(agentSettlementAmount)
|
||||
- 기존 `GetCalculateByCreatorItem`은 건수가 없고 total 객체도 없으므로, 에이전트 전용 응답 DTO는 신규 정의가 필요하다.
|
||||
|
||||
## 구현 방향
|
||||
|
||||
### 1차 구현 범위 (2026-04-09)
|
||||
- 이번 구현 슬라이스는 관리자 전용 기능만 포함한다.
|
||||
- 포함 범위
|
||||
- 에이전트-크리에이터 소속 지정 API
|
||||
- 에이전트-크리에이터 소속 해제 API
|
||||
- 에이전트 정산 비율 생성 API
|
||||
- 에이전트 정산 비율 수정 API
|
||||
- 에이전트 정산 비율 목록 API
|
||||
- 위 기능에 필요한 엔티티/리포지토리/서비스/테스트/DDL 문서
|
||||
- 제외 범위
|
||||
- 에이전트 본인 소속 크리에이터 목록 조회 API
|
||||
- 라이브/콘텐츠/커뮤니티/채널후원/콘텐츠후원 에이전트 정산 조회 API
|
||||
|
||||
### 권장 설계
|
||||
- 공유 도메인 모델/리포지토리는 `kr.co.vividnext.sodalive.partner.agent` 하위 패키지에 둔다.
|
||||
- `ADMIN` 전용 controller/service 진입점은 `kr.co.vividnext.sodalive.admin.partner.agent` 하위 패키지에 둔다.
|
||||
- `AGENT` 전용 정산 조회 진입점은 `kr.co.vividnext.sodalive.partner.agent.calculate` 하위 패키지에 둔다.
|
||||
- 기존 `admin.calculate`, `creator.admin.calculate`, `admin.member`의 구현 패턴은 재사용하되, DTO/쿼리/서비스는 에이전트 요구사항에 맞는 별도 모듈로 분리한다.
|
||||
- 에이전트-크리에이터 소속 관계와 에이전트 정산 비율은 전용 엔티티로 분리해 기능 응집도를 유지한다.
|
||||
|
||||
### 3차 구현 범위 (이력형 소속/비율 + 확정 정산 스냅샷)
|
||||
- 이번 구현 슬라이스는 기존 current-state 기반 에이전트 정산 구조를 historical model로 전환한다.
|
||||
- 포함 범위
|
||||
- `agent_creator_relation`을 `assignedAt/unassignedAt` 기반 이력형 소속 모델로 전환
|
||||
- `agent_settlement_ratio`를 `effectiveFrom/effectiveTo` 기반 이력형 비율 모델로 전환
|
||||
- 관리자 소속 지정/해제 API를 시간 경계 기반으로 수정
|
||||
- 관리자 비율 생성/수정/조회 API를 이력형 비율 기준으로 수정
|
||||
- AGENT 정산 조회 쿼리를 거래 시점(event time)의 소속/비율을 기준으로 계산하도록 수정
|
||||
- 확정 정산 스냅샷 저장 모델 및 관리자 확정 API 추가
|
||||
- finalized 기간 조회는 스냅샷 우선, 미확정 기간 조회는 live 계산 유지
|
||||
- 위 구조에 필요한 테스트, DDL 문서, 계획 문서 갱신
|
||||
- 제외 범위
|
||||
- 기존 과거 데이터의 완전 복원 보장
|
||||
- 이벤트소싱 도입
|
||||
- 프론트엔드 화면 개편
|
||||
|
||||
### current-state 설계의 한계
|
||||
- 현재 `AgentCreatorRelation`은 `agent`, `creator`만 저장하고 remove 시 hard-delete 하므로 과거 소속 이력을 보존하지 못한다.
|
||||
- 현재 `AgentSettlementRatio`는 `deletedAt` 기반 현재 활성 행 조회에 의존하므로 특정 거래 시점의 비율을 안정적으로 재현하지 못한다.
|
||||
- 현재 `AgentCalculateQueryRepository`는 거래 발생 시각(`useCan.createdAt`, `order.createdAt`)으로 기간을 자르면서도, 소속/비율은 현재 row를 기준으로 조인한다.
|
||||
- 따라서 크리에이터가 에이전트에서 해제되거나 비율이 변경되면, 과거 정산 조회 결과도 바뀔 수 있다.
|
||||
- 사용자가 요구한 "당시 적용 비율까지 고정된 완전한 과거 정산"은 current-state 조인만으로 충족할 수 없다.
|
||||
|
||||
### 변경 후 목표 모델
|
||||
- 소속 관계는 append-only 이력 모델로 관리한다.
|
||||
- `assignedAt`: 소속 시작 시각
|
||||
- `unassignedAt`: 소속 종료 시각(nullable)
|
||||
- 현재 소속은 `unassignedAt is null`로 정의한다.
|
||||
- 정산 비율도 append-only 이력 모델로 관리한다.
|
||||
- `effectiveFrom`: 비율 시작 시각
|
||||
- `effectiveTo`: 비율 종료 시각(nullable)
|
||||
- 현재 비율은 `effectiveTo is null`로 정의한다.
|
||||
- 정산 확정 시점에는 별도 스냅샷 테이블에 결과를 저장한다.
|
||||
- 소속/비율 foreign key만 저장하지 않고, 적용된 비율과 계산 결과를 숫자 값으로 함께 저장한다.
|
||||
- 이후 소속/비율 변경이 발생해도 확정 정산은 변경되지 않는다.
|
||||
|
||||
### 확정 정산 스냅샷 설계 초안
|
||||
- 신규 도메인 패키지 후보: `kr.co.vividnext.sodalive.partner.agent.settlement.snapshot`
|
||||
- 신규 관리자 진입점 패키지 후보: `kr.co.vividnext.sodalive.admin.partner.agent.settlement`
|
||||
- 신규 스냅샷 엔티티 초안 필드
|
||||
- `periodStart`, `periodEnd`
|
||||
- `settlementType` (`LIVE`, `CONTENT`, `COMMUNITY`, `CHANNEL_DONATION`, `CONTENT_DONATION`)
|
||||
- `agentId`, `agentNickname`
|
||||
- `creatorId`, `creatorNickname`
|
||||
- `assignmentId`
|
||||
- `agentSettlementRatioId`, `appliedAgentSettlementRatio`
|
||||
- `count`, `totalCan`, `krw`, `fee`, `settlementAmount`, `agentSettlementAmount`
|
||||
- 필요 시 `tax`, `depositAmount`
|
||||
- `finalizedAt`, `finalizedByMemberId`
|
||||
- 스냅샷은 append-only로 저장하고 동일 기간/타입/agent/creator 기준 재확정은 idempotent하게 막거나 재사용한다.
|
||||
|
||||
### 조회 전략 변경
|
||||
- finalized 기간 조회
|
||||
- 스냅샷 데이터가 있으면 스냅샷을 우선 조회한다.
|
||||
- 스냅샷 데이터는 이후 소속 해제/비율 변경과 무관하게 그대로 반환한다.
|
||||
- 미확정 기간 조회
|
||||
- `AgentCalculateQueryRepository`에서 거래 시점 기준으로 소속/비율 이력 row를 찾아 계산한다.
|
||||
- 시간 경계는 `start <= txTime < end` 형태의 반열린 구간으로 처리한다.
|
||||
|
||||
### API 변경 방향
|
||||
- 관리자 소속 지정 API
|
||||
- 기존 path 유지
|
||||
- request에 `assignedAt` 추가
|
||||
- 관리자 소속 해제 API
|
||||
- 기존 path 유지
|
||||
- request에 `unassignedAt` 추가
|
||||
- delete 대신 종료 시각 기록
|
||||
- 관리자 비율 생성/수정 API
|
||||
- 기존 path 유지
|
||||
- request에 `effectiveFrom` 추가
|
||||
- update는 기존 행 수정이 아니라 이전 활성 행 종료 + 신규 행 추가로 동작
|
||||
- 관리자 확정 정산 API
|
||||
- 신규 path 후보: `POST /admin/partner/agent/settlement/finalize`
|
||||
- 입력: 기간, 정산 타입, 대상 에이전트(`agentId` 기준 단일 에이전트 확정)
|
||||
- 에이전트 조회 API
|
||||
- 기존 path 유지
|
||||
- finalized 기간은 스냅샷 우선, 그 외 기간은 live 계산
|
||||
|
||||
### 패키지/파일 초안
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/assignment/AgentCreatorRelation.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/assignment/AgentCreatorRelationRepository.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/assignment/AdminAgentCreatorController.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/assignment/AdminAgentCreatorService.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/assignment/AssignAgentCreatorRequest.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/assignment/RemoveAgentCreatorRequest.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/ratio/AgentSettlementRatio.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/ratio/AgentSettlementRatioRepository.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/ratio/AdminAgentSettlementRatioController.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/ratio/AdminAgentSettlementRatioService.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateController.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateService.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateQueryRepository.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/response/*`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/settlement/snapshot/AgentSettlementSnapshot.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/settlement/snapshot/AgentSettlementSnapshotRepository.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/settlement/AdminAgentSettlementSnapshotController.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/settlement/AdminAgentSettlementSnapshotService.kt`
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/**`
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/**`
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/settlement/**`
|
||||
- `docs/20260408_에이전트권한및정산기능추가.md`
|
||||
|
||||
### API 방향
|
||||
- 관리자 전용
|
||||
- 크리에이터 소속 지정 API
|
||||
- 크리에이터 소속 해제 API
|
||||
- 에이전트 정산 비율 생성/수정/조회 API
|
||||
- 에이전트 전용
|
||||
- 소속 크리에이터 목록 조회 API
|
||||
- 라이브 크리에이터별 현황 조회 API
|
||||
- 콘텐츠 크리에이터별 현황 조회 API
|
||||
- 커뮤니티 크리에이터별 현황 조회 API
|
||||
- 채널후원 크리에이터별 현황 조회 API
|
||||
- 콘텐츠후원 크리에이터별 현황 조회 API
|
||||
|
||||
### 역할별 신규 엔드포인트 정리
|
||||
|
||||
#### ADMIN 전용 엔드포인트
|
||||
| Method | Path | 권한 | 설명 |
|
||||
|---|---|---|---|
|
||||
| `POST` | `/admin/partner/agent/assignment` | `ADMIN` | 에이전트와 크리에이터 소속을 지정한다. |
|
||||
| `POST` | `/admin/partner/agent/assignment/remove` | `ADMIN` | 지정된 크리에이터의 에이전트 소속을 해제한다. |
|
||||
| `POST` | `/admin/partner/agent/ratio` | `ADMIN` | 에이전트 정산 비율을 생성한다. |
|
||||
| `POST` | `/admin/partner/agent/ratio/update` | `ADMIN` | 기존 에이전트 정산 비율을 수정한다. |
|
||||
| `GET` | `/admin/partner/agent/ratio` | `ADMIN` | 에이전트 정산 비율 목록을 페이지 단위로 조회한다. |
|
||||
|
||||
#### AGENT 전용 엔드포인트
|
||||
| Method | Path | 권한 | 설명 |
|
||||
|---|---|---|---|
|
||||
| `GET` | `/agent/calculate/creator/list` | `AGENT` | 로그인한 에이전트에게 소속된 크리에이터 목록을 조회한다. |
|
||||
| `GET` | `/agent/calculate/live-by-creator` | `AGENT` | 소속 크리에이터의 라이브 정산 현황을 크리에이터별로 조회한다. |
|
||||
| `GET` | `/agent/calculate/content-by-creator` | `AGENT` | 소속 크리에이터의 콘텐츠 판매 정산 현황을 크리에이터별로 조회한다. |
|
||||
| `GET` | `/agent/calculate/community-by-creator` | `AGENT` | 소속 크리에이터의 커뮤니티 정산 현황을 크리에이터별로 조회한다. |
|
||||
| `GET` | `/agent/calculate/channel-donation-by-creator` | `AGENT` | 소속 크리에이터의 채널후원 정산 현황을 크리에이터별로 조회한다. |
|
||||
| `GET` | `/agent/calculate/content-donation-by-creator` | `AGENT` | 소속 크리에이터의 콘텐츠후원 정산 현황을 크리에이터별로 조회한다. |
|
||||
|
||||
#### 공통 인증 메모
|
||||
- 관리자 엔드포인트는 모두 `@PreAuthorize("hasRole('ADMIN')")` 기준으로 제한한다.
|
||||
- 에이전트 엔드포인트는 모두 `@PreAuthorize("hasRole('AGENT')")` 기준으로 제한한다.
|
||||
- 에이전트 조회 API는 `@AuthenticationPrincipal(... ) member: Member?`를 통해 로그인 사용자를 확인하고, `agent_creator_relation` 기준으로 본인 소속 크리에이터 데이터만 조회한다.
|
||||
|
||||
### 패키지/엔드포인트 배치에 대한 최종 판단
|
||||
|
||||
#### 1. ADMIN 전용 partner-agent API는 어디에 두는 것이 더 자연스러운가?
|
||||
- **최종 판단: `ADMIN` 전용 controller/service 진입점은 `kr.co.vividnext.sodalive.admin.partner.agent.*` 아래로 옮기는 것이 더 자연스럽다.**
|
||||
- 근거는 이 저장소의 기존 관례가 관리자 전용 기능을 `kr.co.vividnext.sodalive.admin.*` 아래에 배치하는 흐름이 더 강하기 때문이다.
|
||||
- 예: `admin/calculate/AdminCalculateController.kt`
|
||||
- 예: `admin/marketing/AdminAdMediaPartnerController.kt`
|
||||
- 예: `admin/member/AdminMemberController.kt`
|
||||
- `creator`처럼 역할이 강하게 분리된 영역도 `creator/admin/*` 패턴을 쓰므로, 현재 `partner.agent.assignment.AdminAgentCreatorController`, `partner.agent.ratio.AdminAgentSettlementRatioController`처럼 **도메인 패키지 안에 관리자 전용 진입점이 섞여 있는 구조는 이 저장소 기준으로는 예외에 가깝다.**
|
||||
- 따라서 권장 구조는 아래와 같다.
|
||||
- **공유 도메인 객체/리포지토리**: `kr.co.vividnext.sodalive.partner.agent.*`
|
||||
- **ADMIN 전용 controller/service**: `kr.co.vividnext.sodalive.admin.partner.agent.*`
|
||||
- 즉, `AgentCreatorRelation`, `AgentSettlementRatio`, repository까지 `admin.*`로 옮기는 것이 아니라, **관리자 진입점만 `admin.*`로 재배치**하는 것이 균형이 가장 좋다.
|
||||
|
||||
#### 2. 소속 크리에이터 목록 조회 API가 `calculate` 아래에 있는 것은 맞는 선택인가?
|
||||
- **최종 판단: 현재 요구사항 범위에서는 유지해도 된다.**
|
||||
- 이유는 이 API가 “에이전트 소속 관리용 일반 목록 API”라기보다, **에이전트 정산 화면에서 크리에이터별 현황 조회로 진입하기 위한 보조 목록 API**에 가깝기 때문이다.
|
||||
- 현재 같은 컨트롤러에는 아래처럼 모두 정산/집계성 조회가 함께 모여 있다.
|
||||
- `/agent/calculate/live-by-creator`
|
||||
- `/agent/calculate/content-by-creator`
|
||||
- `/agent/calculate/community-by-creator`
|
||||
- `/agent/calculate/channel-donation-by-creator`
|
||||
- `/agent/calculate/content-donation-by-creator`
|
||||
- 따라서 **현재 맥락에서는 `/agent/calculate/creator/list`를 정산 조회의 진입용 목록으로 보는 해석이 가능하고, 응집도도 유지된다.**
|
||||
- 다만 이 API가 앞으로 정산 외 목적(메시지, 운영, 소속 관리, 일반 대시보드)에도 재사용되기 시작하면, 그 시점에는 아래처럼 분리 재검토하는 것이 맞다.
|
||||
- 패키지 후보: `kr.co.vividnext.sodalive.partner.agent.assignment` 또는 `kr.co.vividnext.sodalive.partner.agent.creator`
|
||||
- 엔드포인트 후보: `/agent/creator/list`, `/agent/assignment/creator/list`
|
||||
|
||||
#### 3. 이번 기능에 대한 권장 정리 기준
|
||||
- **ADMIN 전용 API**: `kr.co.vividnext.sodalive.admin.partner.agent.*`
|
||||
- **AGENT 정산 조회 API**: `kr.co.vividnext.sodalive.partner.agent.calculate.*`
|
||||
- **공유 도메인 모델/리포지토리**: `kr.co.vividnext.sodalive.partner.agent.*`
|
||||
- **`/agent/calculate/creator/list`**: 현재는 유지, 단 정산 외 재사용이 커지면 별도 read/assignment 축으로 분리
|
||||
|
||||
## 작업 체크리스트
|
||||
|
||||
- [x] 기존 `MemberRole.AGENT` 사용 범위를 점검하고, 관리자 화면/API에서 에이전트 계정을 식별할 수 있는 조회 경로를 확정한다.
|
||||
- [x] `kr.co.vividnext.sodalive.partner.agent` 패키지 아래에 에이전트-크리에이터 소속 전용 엔티티/리포지토리를 추가한다.
|
||||
- [x] 소속 엔티티에 `creator_id` 유니크 제약과 agent/creator role 검증 로직을 추가해 "한 크리에이터는 하나의 에이전트에만 소속" 규칙을 보장한다.
|
||||
- [x] 관리자 전용 크리에이터 소속 지정 API를 추가한다.
|
||||
- [x] 관리자 전용 크리에이터 소속 해제 API를 추가한다.
|
||||
- [x] 에이전트 정산 비율 전용 엔티티/리포지토리를 추가하고, 에이전트당 단일 비율만 유지되도록 한다.
|
||||
- [x] 관리자 전용 에이전트 정산 비율 생성/수정/조회 API를 추가한다.
|
||||
- [x] 1차 구현 범위의 assignment/ratio 컨트롤러/서비스 테스트를 추가해 role 검증, 중복 소속 방지, 누락 엔티티, pageable 위임을 검증한다.
|
||||
- [x] 신규 assignment/ratio 테이블 생성을 위한 DDL 문서를 `docs/20260409_partner_agent_assignment_ratio_ddl.sql`에 추가한다.
|
||||
- [x] 에이전트 본인의 소속 크리에이터 목록 조회 API를 추가한다.
|
||||
- [x] `/agent/calculate/creator/list`가 현재 시각 기준 `assignedAt <= now < unassignedAt` 활성 구간의 크리에이터만 노출하도록 보강한다.
|
||||
- [x] 라이브 현황용 agent 전용 Query/DTO/응답을 추가하고, `settlementAmount` 기준 `agentSettlementAmount`를 계산한다.
|
||||
- [x] 콘텐츠 판매 현황용 agent 전용 Query/DTO/응답을 추가하고, 기존 콘텐츠 정산 비율(`audioContent.settlementRatio` 또는 `CreatorSettlementRatio.contentSettlementRatio`)을 재사용한다.
|
||||
- [x] 커뮤니티 현황용 agent 전용 Query/DTO/응답을 추가하고, `CreatorSettlementRatio.communitySettlementRatio` 기반 정산 후 agent 금액을 계산한다.
|
||||
- [x] 채널후원 현황용 agent 전용 Query/DTO/응답을 추가하고, 기존 `ChannelDonationSettlementCalculator` 결과의 `settlementAmount` 기준으로 agent 금액을 계산한다.
|
||||
- [x] 콘텐츠후원 현황용 agent 전용 Query/DTO/응답을 추가하고, 기존 콘텐츠후원 계산 결과의 `settlementAmount` 기준으로 agent 금액을 계산한다.
|
||||
- [x] 5개 현황 응답 모두에 `totalCount + total + items` 구조를 맞추고, item/total 양쪽에 `agentSettlementAmount`를 포함한다.
|
||||
- [x] AGENT 계정이 자신에게 소속된 크리에이터 데이터만 조회하도록 `@AuthenticationPrincipal` + relation 기반 필터링을 구현한다.
|
||||
- [x] ADMIN/AGENT 권한 오류, 잘못된 role 요청, 중복 소속 요청, 비소속 데이터 조회 차단에 대한 예외 처리를 추가한다.
|
||||
- [x] 컨트롤러/서비스/쿼리 리포지토리 테스트를 추가해 소속 제약, 권한 제약, 정산 계산식, total 합계를 검증한다.
|
||||
- [x] `./gradlew test`, `./gradlew build`로 최종 검증한다.
|
||||
- [x] `agent_creator_relation`에 `assignedAt`, `unassignedAt`를 추가하고 current-state 단일 row 모델을 append-only 이력 모델로 전환한다.
|
||||
- [x] 관리자 소속 지정 API가 `assignedAt`을 받아 활성 기간 중복을 검증하도록 수정한다.
|
||||
- [x] 관리자 소속 해제 API가 hard-delete 대신 `unassignedAt` 종료 처리로 변경되도록 수정한다.
|
||||
- [x] `agent_settlement_ratio`에 `effectiveFrom`, `effectiveTo`를 추가하고 단일 현재 row 갱신 모델을 append-only 이력 모델로 전환한다.
|
||||
- [x] 관리자 비율 생성/수정 API가 `effectiveFrom`을 받아 기존 활성 row 종료 + 신규 row 추가로 동작하도록 수정한다.
|
||||
- [x] 관리자 비율 생성/수정 API가 `effectiveFrom` backdate, 동일 시각 입력, 기존 ratio history와 겹치는 시점을 거절하도록 검증을 보강한다.
|
||||
- [x] 관리자 비율 생성/수정 API가 `settlementRatio`를 0..100 범위로 검증하도록 보강한다.
|
||||
- [x] `agent_settlement_ratio` DDL에 MySQL 생성 컬럼 + UNIQUE 인덱스로 active row 단일성을 보장하고, `effective_from < effective_to` 기간 무결성 제약을 추가한다.
|
||||
- [x] `agent_creator_relation` DDL에 MySQL 생성 컬럼 + UNIQUE 인덱스로 active row 단일성을 보장하고, `assigned_at < unassigned_at` 기간 무결성 제약을 추가한다.
|
||||
- [x] 관리자 소속/비율 쓰기 경로가 `MemberRepository.findByIdForUpdate(...)` 기반 비관적 락과 unique violation 대응 패턴으로 직렬화되도록 보강한다.
|
||||
- [x] AGENT 정산 조회가 거래 시점 기준의 소속 이력과 비율 이력을 조인하도록 `AgentCalculateQueryRepository`를 수정한다.
|
||||
- [x] 기간 중 소속 변경 또는 비율 변경이 있는 경우 결과가 올바르게 분리/집계되는 테스트를 추가한다.
|
||||
- [x] AGENT 정산 조회의 paged query가 사전 조회된 `creatorIds`가 빈 페이지일 때 전체 결과로 fallback하지 않고 빈 rows/items를 반환하도록 보강한다.
|
||||
- [x] AGENT 정산 조회에서 `agent_settlement_ratio` 이력이 없으면 agent 정산금을 0% 대신 10% 기본값으로 계산하도록 수정한다.
|
||||
- [x] 확정 정산 스냅샷 엔티티/리포지토리/관리자 API를 추가한다.
|
||||
- [x] 확정 정산 스냅샷이 소속/비율 foreign key뿐 아니라 적용 비율과 계산 결과 숫자값을 함께 저장하도록 구현한다.
|
||||
- 정정(2026-04-09): 현재 구현은 `appliedAgentSettlementRatio`와 계산 결과 숫자값은 저장하지만, 설계 초안에 명시한 `assignmentId`, `agentSettlementRatioId`는 아직 스냅샷/DDL에 포함하지 않았다. 따라서 이 항목은 엄밀히는 부분 충족 상태로 본다.
|
||||
- [x] `AgentSettlementSnapshot` 및 `agent_settlement_snapshot` DDL에 `assignmentId`, `agentSettlementRatioId` 컬럼을 추가한다.
|
||||
- [x] `AgentCalculateQueryRepository`와 snapshot 생성용 query DTO에 거래 시점 기준 `assignmentId`, `agentSettlementRatioId` projection을 추가한다.
|
||||
- [x] `AdminAgentSettlementSnapshotService`와 관련 테스트를 갱신해 finalize 시점에 foreign key + 적용 비율 + 계산 숫자값이 함께 저장되도록 보완한다.
|
||||
- 참고: 위 보완은 creator-period summary가 단일 소속/단일 비율 이력 row로 귀결되는 경우의 추적성은 복구하지만, 기간 중 복수 history row가 섞인 summary의 완전 provenance까지 보장하지는 않는다. 그 수준의 감사 추적이 필요하면 별도 snapshot source detail 테이블이 추가로 필요하다.
|
||||
- [x] `agent_settlement_snapshot_source_detail`(가칭) DDL을 추가해 summary를 구성한 원천 source row별 provenance를 별도 저장한다.
|
||||
- [x] finalize가 `raw source row`를 기준으로 source detail과 creator-period summary를 같은 트랜잭션 안에서 함께 저장하도록 보강한다.
|
||||
- [x] source detail이 1건인 summary만 `assignmentId`, `agentSettlementRatioId`, `appliedAgentSettlementRatio`를 채우고, mixed-period summary는 `null`로 유지하는 규칙을 테스트로 고정한다.
|
||||
- [x] finalized 기간 조회는 스냅샷 우선, 미확정 기간 조회는 live 계산을 사용하도록 분기한다.
|
||||
- [x] 신규 이력/스냅샷 구조에 맞는 DDL 문서를 추가 또는 기존 DDL 문서를 확장한다.
|
||||
- [x] 기존 계획 문서 하단 검증 기록에 이력형 전환과 스냅샷 도입 구현/검증 결과를 누적한다.
|
||||
|
||||
## 세부 구현 메모
|
||||
|
||||
### 1. 소속 관계 구현 기준
|
||||
- 기존 `Member` 엔티티에 agent 필드를 직접 추가하지 않고, 전용 relation 테이블로 구현한다.
|
||||
- 이유는 다음과 같다.
|
||||
- 에이전트 전용 기능을 `partner.agent` 패키지에 응집시킬 수 있다.
|
||||
- creator role 제약과 unique 제약을 명확히 걸 수 있다.
|
||||
- 향후 소속 이력/상태 필드가 필요해져도 확장이 쉽다.
|
||||
|
||||
### 2. 정산 비율 구현 기준
|
||||
- 기존 `CreatorSettlementRatio`가 도메인별 다중 비율을 관리하므로, 에이전트는 별도 `AgentSettlementRatio`로 분리하는 편이 자연스럽다.
|
||||
- 필드는 단일 `settlementRatio`만 두고, 대상 회원은 `MemberRole.AGENT`로 제한한다.
|
||||
|
||||
### 3. 조회 응답 구현 기준
|
||||
- 라이브/콘텐츠/커뮤니티의 기존 관리자 creator별 응답은 count/total 객체가 부족하므로 재사용보다 agent 전용 응답 신설이 적합하다.
|
||||
- 채널후원 응답은 이미 `total + items` 구조가 있어 이를 가장 가까운 기준으로 삼는다.
|
||||
- 에이전트 응답은 아래 공통 필드를 기준으로 통일한다.
|
||||
- `creatorId`
|
||||
- `creatorNickname`
|
||||
- `count`
|
||||
- `totalCan`
|
||||
- `krw`
|
||||
- `fee`
|
||||
- `settlementAmount`
|
||||
- `agentSettlementAmount`
|
||||
- 필요 시 `tax`, `depositAmount`
|
||||
|
||||
### 4. 재사용 기준 파일
|
||||
- 권한/인증 패턴: `AdminMemberController`, `CreatorAdminCalculateController`, `SecurityConfig`
|
||||
- creator별 집계 패턴: `AdminCalculateQueryRepository`
|
||||
- 채널후원 total 응답 패턴: `AdminChannelDonationCalculateService`, `GetAdminChannelDonationSettlementTotal`
|
||||
- 정산 비율 검증 패턴: `CreatorSettlementRatioService`
|
||||
|
||||
### 5. 후속 보완 구현 순서 (2026-04-09 추가)
|
||||
- 아래 보완 작업은 **ratio 입력 무결성 차단 → 관리자 쓰기 직렬화/DDL 보강 → snapshot traceability 연결 → 전체 검증** 순서로 진행한다.
|
||||
|
||||
#### 5-1. ratio 입력 무결성 차단부터 먼저 수정한다.
|
||||
- [x] `src/test/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/ratio/AgentSettlementRatioServiceTest.kt`에 아래 RED 테스트를 추가한다.
|
||||
- [x] active row보다 과거 `effectiveFrom`으로 create/update 요청 시 예외가 발생한다.
|
||||
- [x] active row와 같은 `effectiveFrom`으로 create/update 요청 시 예외가 발생한다.
|
||||
- [x] active row가 없더라도 기존 closed history와 겹치는 `effectiveFrom`이면 예외가 발생한다.
|
||||
- [x] `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/ratio/AdminAgentSettlementRatioService.kt`에서 `effectiveFrom` backdate / same-time / history overlap을 거절하도록 검증을 추가한다.
|
||||
- [x] `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/ratio/AgentSettlementRatioRepository.kt`에 history overlap 판별용 조회 메서드를 추가한다.
|
||||
- [x] ratio 서비스 테스트를 재실행해 backdate 차단이 먼저 보장되는지 확인한다.
|
||||
|
||||
#### 5-2. 관리자 소속/비율 쓰기 경로를 직렬화한다.
|
||||
- [x] `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/ratio/AdminAgentSettlementRatioService.kt`에서 agent 대상 `MemberRepository.findByIdForUpdate(...)`를 사용하도록 수정한다.
|
||||
- [x] `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/assignment/AdminAgentCreatorService.kt`에서 creator 대상 `MemberRepository.findByIdForUpdate(...)`를 사용하도록 수정한다.
|
||||
- [x] 두 서비스 모두 `saveAndFlush` + `DataIntegrityViolationException` 대응 패턴으로 unique violation을 사용자 예외로 변환하도록 보강한다.
|
||||
- [x] `src/test/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/assignment/AdminAgentCreatorServiceTest.kt`, `src/test/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/ratio/AgentSettlementRatioServiceTest.kt`에 락/unique violation 대응 케이스를 추가한다.
|
||||
|
||||
#### 5-3. active row 단일성과 기간 무결성을 DDL/엔티티에 맞춘다.
|
||||
- [x] `docs/20260409_partner_agent_assignment_ratio_ddl.sql` 기준으로 `agent_creator_relation`, `agent_settlement_ratio`, `agent_settlement_snapshot` 최종 스키마가 구현 코드와 일치하는지 다시 점검한다.
|
||||
- [x] `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/assignment/AgentCreatorRelation.kt`, `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/ratio/AgentSettlementRatio.kt`의 매핑이 DDL의 최종 컬럼 구조와 충돌하지 않는지 확인한다.
|
||||
- [x] generated column(`active_creator_key`, `active_ratio_key`)은 JPA 쓰기 대상에서 제외하고, 서비스/리포지토리 로직이 해당 컬럼 없이도 동작하는지 확인한다.
|
||||
|
||||
#### 5-4. snapshot traceability를 query → service → entity 순서로 연결한다.
|
||||
- [x] `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/settlement/snapshot/AgentSettlementSnapshot.kt`에 `assignmentId`, `agentSettlementRatioId` 필드를 추가한다.
|
||||
- [x] `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateQueryRepository.kt`와 snapshot 생성용 query DTO에 거래 시점 기준 `assignmentId`, `agentSettlementRatioId` projection을 추가한다.
|
||||
- [x] `docs/20260409_partner_agent_assignment_ratio_ddl.sql`에 `agent_settlement_snapshot_source_detail`(가칭) 테이블을 추가하고, `snapshot_id`, `assignment_id`, `agent_settlement_ratio_id`, source subtotal 컬럼, 조회 인덱스/FK를 정의한다.
|
||||
- [x] `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/settlement/snapshot/**`에 source detail 엔티티/리포지토리 파일을 추가한다.
|
||||
- [x] `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/settlement/AdminAgentSettlementSnapshotService.kt`를 `raw source row` 기준의 source detail과 creator-period summary를 함께 저장하도록 바꾼다.
|
||||
- [x] creator-period summary가 단일 source row로 귀결될 때만 `assignmentId`, `agentSettlementRatioId`, `appliedAgentSettlementRatio`를 summary에 채우고, mixed-period summary는 `null`로 저장하도록 매핑 규칙을 고정한다.
|
||||
- [x] `src/test/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/settlement/AdminAgentSettlementSnapshotServiceTest.kt`에 아래 검증을 추가한다.
|
||||
- [x] 단일 source summary는 summary row와 detail row가 같은 `assignmentId`, `agentSettlementRatioId`, `appliedAgentSettlementRatio`를 가진다.
|
||||
- [x] mixed-period summary는 summary의 `assignmentId`, `agentSettlementRatioId`, `appliedAgentSettlementRatio`가 `null`이고, detail row 여러 건으로 provenance를 복원할 수 있다.
|
||||
- [x] detail 합계가 summary 숫자값과 일치한다.
|
||||
|
||||
#### 5-5. 최종 회귀 검증을 수행한다.
|
||||
- [x] `./gradlew test --tests "kr.co.vividnext.sodalive.admin.partner.agent.assignment.*"`
|
||||
- [x] `./gradlew test --tests "kr.co.vividnext.sodalive.admin.partner.agent.ratio.*"`
|
||||
- [x] `./gradlew test --tests "kr.co.vividnext.sodalive.admin.partner.agent.settlement.*"`
|
||||
- [x] `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.calculate.*"`
|
||||
- [x] `./gradlew test`
|
||||
- [x] `./gradlew build`
|
||||
|
||||
## 검증 계획
|
||||
- 단위/서비스 테스트
|
||||
- agent와 creator role 검증
|
||||
- creator 중복 소속 방지
|
||||
- 비소속 creator 조회 차단
|
||||
- agentSettlementAmount 반올림 계산 검증
|
||||
- category별 total 합계 검증
|
||||
- 통합 성격 검증
|
||||
- ADMIN assignment API 정상/실패 케이스
|
||||
- AGENT 목록/정산 조회 API 정상/권한 실패 케이스
|
||||
- 빌드 검증
|
||||
- `./gradlew test`
|
||||
- `./gradlew build`
|
||||
|
||||
---
|
||||
|
||||
## 검증 기록
|
||||
|
||||
### 계획 수립 1차
|
||||
- 무엇을: 에이전트 권한, 소속 관계, 관리자 소속 관리 API, 에이전트 정산 비율, 에이전트 전용 크리에이터별 정산 조회 기능의 구현 범위를 분석하고 작업 계획 문서를 작성했다.
|
||||
- 왜: 기존 코드베이스에 이미 존재하는 `AGENT` 역할, 관리자 정산 모듈, 크리에이터 정산 비율 모듈을 기준으로 중복 구현 없이 가장 자연스러운 확장 경로를 먼저 확정하기 위해서다.
|
||||
- 어떻게:
|
||||
- 권한/역할/정산 패턴을 코드 기준으로 확인했다.
|
||||
- `docs` 폴더의 기존 작업 계획 문서 형식을 확인해 동일한 형식으로 문서를 작성했다.
|
||||
- 실행/확인 결과:
|
||||
- `explore` 백그라운드 탐색 2건(권한 패턴, 정산 패턴) → 완료
|
||||
- `read`로 확인한 기준 파일: `Member.kt`, `AdminMemberController.kt`, `AdminMemberService.kt`, `SecurityConfig.kt`, `AdminCalculateController.kt`, `AdminCalculateService.kt`, `AdminCalculateQueryRepository.kt`, `CreatorAdminCalculateController.kt`, `CreatorAdminCalculateService.kt`, `AdminChannelDonationCalculateController.kt`, `AdminChannelDonationCalculateService.kt`, `ChannelDonationSettlementCalculator.kt`
|
||||
- `./gradlew test` / `./gradlew build` → 문서 작성 단계이므로 미실행
|
||||
|
||||
### 1차 구현 1차
|
||||
- 무엇을: 공유 도메인인 `partner.agent.assignment`, `partner.agent.ratio`와 관리자 진입점인 `admin.partner.agent.assignment`, `admin.partner.agent.ratio` 패키지에 관리자 전용 assignment/remove, ratio create/update/list 기능과 테스트, 신규 테이블 DDL 문서를 추가했다.
|
||||
- 왜: 전체 에이전트 기능 중 첫 vertical slice로서 관리자 관점의 소속 관리와 정산 비율 관리부터 독립적으로 배포 가능한 최소 단위를 먼저 완성하기 위해서다.
|
||||
- 어떻게:
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/assignment/**`에 relation 엔티티/리포지토리/요청 DTO를 추가했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/assignment/**`에 관리자 전용 서비스/컨트롤러를 추가했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/ratio/**`에 ratio 엔티티/리포지토리(querydsl)/요청·응답 DTO를 추가했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/ratio/**`에 관리자 전용 서비스/컨트롤러를 추가했다.
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/**`에 assignment/ratio 서비스·컨트롤러 테스트를 추가하고 TDD 순서로 RED→GREEN을 확인했다.
|
||||
- `docs/20260409_partner_agent_assignment_ratio_ddl.sql`에 `agent_creator_relation`, `agent_settlement_ratio` 생성 스크립트를 추가했다.
|
||||
- 실행/확인 결과:
|
||||
- `lsp_diagnostics` on `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent`, `src/test/kotlin/kr/co/vividnext/sodalive/partner/agent` → 불가 (현재 환경에 `.kt`용 LSP 서버 미구성)
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.assignment.*"` → 1차 실행 실패(신규 클래스 unresolved reference), 구현 후 재실행 성공
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.ratio.*"` → 1차 실행 실패(신규 클래스 unresolved reference), 구현 후 재실행 성공
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.*"` → 성공
|
||||
- `./gradlew test` → 성공
|
||||
- `./gradlew build` → 성공
|
||||
|
||||
### 2차 구현 1차
|
||||
- 무엇을: `partner.agent.calculate` 패키지 아래에 AGENT 전용 소속 크리에이터 목록 조회 API와 라이브/콘텐츠/커뮤니티/채널후원/콘텐츠후원 creator-level summary API, QueryRepository, 응답 DTO, 테스트를 추가했다.
|
||||
- 왜: 에이전트 기능의 두 번째 vertical slice로서 실제 에이전트 계정이 본인에게 배정된 크리에이터 범위 안에서만 정산 현황을 볼 수 있도록 기능을 완성하기 위해서다.
|
||||
- 어떻게:
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/**`에 `AgentCalculateController`, `AgentCalculateService`, `AgentCalculateQueryRepository`, assigned creator 응답 DTO, 일반 정산 summary DTO, 채널후원 summary DTO를 추가했다.
|
||||
- AGENT 인증 패턴은 `@PreAuthorize("hasRole('AGENT')")`와 `@AuthenticationPrincipal(expression = "#this == 'anonymousUser' ? null : member") member: Member?` 가드절로 맞췄다.
|
||||
- QueryRepository는 `agent_creator_relation` 조인으로 소속 크리에이터만 필터링하고, 콘텐츠 summary는 creator별 페이지를 유지하면서 콘텐츠별 정산 비율 버킷을 병합하도록 구현했다.
|
||||
- `agentSettlementAmount`는 모든 응답에서 creator의 세전 `settlementAmount` 기준으로 `round(settlementAmount * agentRatio / 100)`를 적용했고, creator `settlementAmount`/`depositAmount` 자체는 차감하지 않았다.
|
||||
- 스키마 변경은 없어서 추가 DDL 문서는 만들지 않았다.
|
||||
- 실행/확인 결과:
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.calculate.*"` → 1차 실행 실패(신규 클래스 unresolved reference), 구현 후 재실행 성공
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.*"` → 성공
|
||||
- `lsp_diagnostics` on `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate`, `src/test/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate` → 불가 (현재 환경에 `.kt`용 LSP 서버 미구성)
|
||||
- `./gradlew test` → 성공
|
||||
- `./gradlew build` → 최초 1회 `ktlintMainSourceSetCheck` 실패(신규 `AgentCalculateService.kt` 줄바꿈 규칙 위반), 포맷 수정 후 재실행 성공
|
||||
|
||||
### 3차 수정
|
||||
- 무엇을: `partner.agent.assignment.*`, `partner.agent.ratio.*` 예외 키를 `SodaMessageSource`에 등록하고, 해당 메시지 조회를 보장하는 단위 테스트를 추가했다.
|
||||
- 왜: 기능 자체는 동작하더라도 메시지 소스에 키가 없으면 런타임에서 사용자에게 의도한 다국어 문구 대신 키 문자열 또는 빈 메시지가 노출될 수 있기 때문이다.
|
||||
- 어떻게:
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/i18n/SodaMessageSourceTest.kt`를 추가해 `partner.agent.assignment.creator_already_assigned`, `partner.agent.ratio.invalid_agent`, `partner.agent.ratio.not_found` 메시지 조회를 RED→GREEN으로 검증했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/i18n/SodaMessageSource.kt`에 `partnerAgentMessages` 맵을 추가하고 `getMessage` 그룹 목록에 포함시켰다.
|
||||
- 실행/확인 결과:
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.i18n.SodaMessageSourceTest"` → 1차 실행 실패(메시지 미등록), 수정 후 재실행 성공
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.*"` → 성공
|
||||
- `./gradlew test` → 성공
|
||||
- `./gradlew build` → 성공
|
||||
- `jshell --class-path "build/classes/kotlin/main:build/resources/main:...kotlin-stdlib..."` 수동 확인 →
|
||||
- `partner.agent.assignment.creator_already_assigned` 한국어 메시지 출력: `이미 다른 에이전트에 소속된 크리에이터입니다.`
|
||||
- `GetAgentCreatorSettlementSummaryQueryData(21, creator-a, 2, 100, 70).toResponseItem(10)` 출력: `agentSettlementAmount=654`
|
||||
- `GetAgentChannelDonationSettlementByCreatorQueryData(21, creator-a, 1, 50).toResponseItem(10)` 출력: `agentSettlementAmount=397`
|
||||
|
||||
### 4차 수정
|
||||
- 무엇을: 관리자 전용 partner-agent controller/service 진입점을 `kr.co.vividnext.sodalive.admin.partner.agent.*` 아래로 재배치했다.
|
||||
- 왜: 이 저장소의 기존 관례상 관리자 전용 진입점은 `admin.*` 계층에 두는 편이 더 일관적이고, 공유 도메인 객체와 관리자 API 진입점을 분리하는 것이 책임 경계를 더 명확하게 만들기 때문이다.
|
||||
- 어떻게:
|
||||
- `AdminAgentCreatorController`, `AdminAgentCreatorService`를 `admin.partner.agent.assignment`로 이동하고, relation/request DTO/repository는 `partner.agent.assignment`에 유지했다.
|
||||
- `AdminAgentSettlementRatioController`, `AdminAgentSettlementRatioService`를 `admin.partner.agent.ratio`로 이동하고, ratio 엔티티/리포지토리/DTO는 `partner.agent.ratio`에 유지했다.
|
||||
- assignment/ratio 테스트 패키지도 `src/test/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/**`로 이동했다.
|
||||
- 계획 문서의 권장 설계/패키지 초안을 실제 구조에 맞게 갱신했다.
|
||||
- 실행/확인 결과:
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.admin.partner.agent.assignment.*"` → 1차 실행 실패(새 패키지 unresolved reference), 이동 후 재실행 성공
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.admin.partner.agent.ratio.*"` → 1차 실행 실패(새 패키지 unresolved reference), 이동 후 재실행 성공
|
||||
- `grep "kr.co.vividnext.sodalive.partner.agent.(assignment|ratio).AdminAgent" src/**/*.kt` 성격 확인 → 코드 기준 잔존 참조 없음
|
||||
- `./gradlew test` → 성공
|
||||
- `./gradlew build` → 성공
|
||||
- `jshell --class-path "build/classes/kotlin/main:build/resources/main:...kotlin-stdlib..."` 수동 확인 →
|
||||
- `kr.co.vividnext.sodalive.admin.partner.agent.assignment.AdminAgentCreatorController` 로딩 성공
|
||||
- `kr.co.vividnext.sodalive.admin.partner.agent.ratio.AdminAgentSettlementRatioController` 로딩 성공
|
||||
- 기존 `kr.co.vividnext.sodalive.partner.agent.assignment.AdminAgentCreatorController` / `...ratio.AdminAgentSettlementRatioController`는 `ClassNotFoundException`으로 미존재 확인
|
||||
|
||||
### 5차 수정
|
||||
- 무엇을: `agent_creator_relation`을 `assignedAt/unassignedAt` 기반 append-only 이력 모델로 전환하고, 관리자 소속 지정/해제 API를 시간 경계 기반 계약으로 수정했다.
|
||||
- 왜: 기존 current-state + hard-delete 구조로는 과거 소속 이력을 보존할 수 없고, 소속 해제 후 재배정 같은 운영 시나리오를 안전하게 표현할 수 없기 때문이다.
|
||||
- 어떻게:
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/assignment/AgentCreatorRelation.kt`에 `assignedAt`, `unassignedAt`를 추가하고 `creator` 연관을 `ManyToOne`으로 변경해 동일 creator의 이력 row 누적을 허용했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/assignment/AssignAgentCreatorRequest.kt`, `RemoveAgentCreatorRequest.kt`에 명시적 시간 필드를 추가하고, `AdminAgentCreatorService.kt`에서 overlap 검증 및 hard-delete 대신 종료 시각 기록으로 동작을 바꿨다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateQueryRepository.kt`에는 현재 동작 유지용으로 `unassignedAt is null` 조건만 추가해 active assignment 조회가 계속 현재 row만 보도록 맞췄다.
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/assignment/AdminAgentCreatorServiceTest.kt`, `AdminAgentCreatorControllerTest.kt`를 TDD로 갱신해 RED에서 새 계약 부재를 확인한 뒤 GREEN으로 구현했다.
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateQueryRepositoryTest.kt`는 새 not-null `assignedAt` 계약에 맞게 relation fixture를 갱신했다.
|
||||
- `docs/20260409_partner_agent_assignment_ratio_ddl.sql`에 `agent_creator_relation` 생성 스키마를 이력형 컬럼/인덱스로 바꾸고, 기존 테이블에 대한 `assigned_at`, `unassigned_at`, unique index 제거, backfill migration 블록을 추가했다.
|
||||
- 실행/확인 결과:
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.admin.partner.agent.assignment.*"` → 1차 실행 실패(새 `assignedAt/unassignedAt`, repository 메서드, 엔티티 필드 부재), 구현 후 재실행 성공
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.admin.partner.agent.assignment.*" --tests "kr.co.vividnext.sodalive.partner.agent.calculate.AgentCalculateQueryRepositoryTest" --tests "kr.co.vividnext.sodalive.i18n.SodaMessageSourceTest"` → 성공
|
||||
- `lsp_diagnostics` on modified Kotlin files/directories → 불가 (현재 환경에 `.kt`용 LSP 서버 미구성)
|
||||
- `./gradlew test`와 `./gradlew build`를 병렬 실행 → 실패 (`build/test-results/test/*.xml` 동시 쓰기 충돌)
|
||||
- `./gradlew test` 순차 재실행 → 성공
|
||||
- `./gradlew build` 순차 재실행 → 성공
|
||||
|
||||
### 6차 수정
|
||||
- 무엇을: `agent_settlement_ratio`를 `effectiveFrom/effectiveTo` 기반 append-only 이력 모델로 전환하고, 관리자 비율 생성/수정/목록 API 계약을 유효 기간 노출 방식으로 갱신했다.
|
||||
- 왜: 기존 `deletedAt` + 단일 current-row 갱신 방식으로는 과거 비율 이력을 보존할 수 없어서, 이후 거래 시점 기준 정산 조회나 운영 감사 시나리오에 필요한 근거 데이터를 남길 수 없기 때문이다.
|
||||
- 어떻게:
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/ratio/AgentSettlementRatioServiceTest.kt`, `AdminAgentSettlementRatioControllerTest.kt`를 먼저 수정해 `effectiveFrom` 입력, `effectiveFrom/effectiveTo` 응답, 기존 활성 row 종료 + 신규 row 추가 동작을 RED로 만들었다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/ratio/AgentSettlementRatio.kt`를 `effectiveFrom/effectiveTo` 필드와 `close(...)` 메서드를 가진 이력 엔티티로 바꾸고, `member` 연관을 `ManyToOne`으로 변경해 동일 agent의 다중 이력 row를 허용했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/ratio/CreateAgentSettlementRatioRequest.kt`, `GetAgentSettlementRatioResponse.kt`, `AgentSettlementRatioRepository.kt`를 갱신해 요청/응답 계약과 active lookup 메서드 `findFirstByMemberIdAndEffectiveToIsNull(...)`를 도입했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/ratio/AdminAgentSettlementRatioService.kt`에서 create/update 모두 기존 활성 row를 `effectiveTo`로 닫은 뒤 새 row를 저장하도록 수정했고, `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateService.kt`는 현재 활성 비율 lookup만 새 repository 메서드로 맞췄다.
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateServiceTest.kt` fixture도 새 repository 메서드와 `effectiveFrom` 필수 생성자에 맞게 최소 호환 수정했다.
|
||||
- `docs/20260409_partner_agent_assignment_ratio_ddl.sql`에는 `agent_settlement_ratio` 생성 스키마를 `effective_from/effective_to` + history index 구조로 변경하고, 기존 테이블에 대한 컬럼 추가/backfill/unique index 제거/`deleted_at` 제거 migration 블록을 확장했다.
|
||||
- 실행/확인 결과:
|
||||
- `./gradlew test --tests kr.co.vividnext.sodalive.admin.partner.agent.ratio.AgentSettlementRatioServiceTest --tests kr.co.vividnext.sodalive.admin.partner.agent.ratio.AdminAgentSettlementRatioControllerTest` → 1차 실행 실패(새 `effectiveFrom/effectiveTo` 계약, repository 메서드, 엔티티 필드 부재), 구현 후 재실행 성공
|
||||
- `lsp_diagnostics` on modified Kotlin files/directories → 불가 (현재 환경에 `.kt`용 LSP 서버 미구성)
|
||||
- `./gradlew test --tests kr.co.vividnext.sodalive.admin.partner.agent.ratio.AgentSettlementRatioServiceTest --tests kr.co.vividnext.sodalive.admin.partner.agent.ratio.AdminAgentSettlementRatioControllerTest --tests kr.co.vividnext.sodalive.partner.agent.calculate.AgentCalculateServiceTest` → 성공
|
||||
- `./gradlew build` → 성공
|
||||
|
||||
### 7차 수정
|
||||
- 무엇을: `AgentCalculateQueryRepository`, `AgentCalculateService`, agent calculate 응답용 query DTO와 테스트를 이벤트 시점 기준 소속/agent ratio 계산 방식으로 수정해, 기간 중 재배정과 agent 비율 변경이 있어도 다섯 개 정산 카테고리의 creator-level 결과가 당시 기준으로 집계되도록 바꿨다.
|
||||
- 왜: 기존 구현은 거래 발생 시각으로 기간만 자르고, 소속은 현재 active relation, agent 비율은 현재 active ratio 한 개를 전체 기간에 적용하고 있어서 중간 재배정/비율 변경이 생기면 과거 조회 결과가 잘못 왜곡됐기 때문이다.
|
||||
- 어떻게:
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateQueryRepositoryTest.kt`에 기간 중 소속 변경, 기간 중 agent ratio 변경을 재현하는 통합 성격 테스트를 먼저 추가해 RED를 확인했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateQueryRepository.kt`에서 라이브/콘텐츠/커뮤니티/채널후원/콘텐츠후원 모두에 대해 거래 시각 기준 `assignedAt <= eventTime < unassignedAt`, `effectiveFrom <= eventTime < effectiveTo` 조건으로 이력 row를 조인하도록 수정했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/GetAgentCreatorSettlementSummaryQueryData.kt`, `GetAgentChannelDonationSettlementByCreatorResponse.kt`, `AgentCalculateService.kt`를 바꿔 row별 agent ratio를 응답 아이템 변환 시 적용하고, 채널후원 포함 전 카테고리에서 creator 기준 merge 후 total을 계산하도록 정리했다.
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateServiceTest.kt`도 row별 agent ratio 기대값으로 갱신해 서비스 레벨 병합 규칙을 검증했다.
|
||||
- 실행/확인 결과:
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.calculate.AgentCalculateQueryRepositoryTest"` → 1차 실행 실패(기간 중 소속 변경/agent ratio 변경 2건 assertion failure), 구현 후 재실행 성공
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.calculate.*"` → 성공
|
||||
- `lsp_diagnostics` on modified Kotlin files → 불가 (현재 환경에 `.kt`용 LSP 서버 미구성)
|
||||
- `./gradlew test` → 성공
|
||||
- `./gradlew build` → 1차 `ktlintTestSourceSetCheck` 실패(신규 테스트 장문 라인), 2차 `ktlintMainSourceSetCheck` 실패(import 순서/장문 line), 포맷 수정 후 재실행 성공
|
||||
|
||||
### 8차 수정
|
||||
- 무엇을: `partner.agent.settlement.snapshot` 패키지에 immutable creator-level snapshot 저장 모델과 repository/request-response mapper를 추가하고, `admin.partner.agent.settlement`에 확정 API를 만들었으며, `AgentCalculateService`가 finalized 기간이면 스냅샷을 우선 읽도록 다섯 카테고리 전체를 연결했다.
|
||||
- 왜: 이력형 소속/비율 계산만으로는 확정 시점의 creator-level 결과를 별도 보존하거나 재사용할 수 없어서, 이후 읽기에서 동일 기간을 다시 계산하지 않고도 확정된 숫자값을 안정적으로 반환할 수 있어야 했기 때문이다.
|
||||
- 어떻게:
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/settlement/AdminAgentSettlementSnapshotServiceTest.kt`, `AdminAgentSettlementSnapshotControllerTest.kt`, `src/test/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateServiceTest.kt`에 snapshot 생성/idempotency/finalized snapshot-first read RED 테스트를 먼저 추가했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/settlement/snapshot/**`에 `AgentSettlementSnapshot`, `AgentSettlementSnapshotRepository`, `FinalizeAgentSettlementSnapshotRequest/Response`, snapshot-to-response mapper를 추가했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/settlement/AdminAgentSettlementSnapshotService.kt`에서 기존 `AgentCalculateQueryRepository` live 계산 결과를 creator-level 응답으로 병합한 뒤 숫자값을 그대로 스냅샷 row에 저장하고, 동일 기간/타입/agent 조합은 `exists...` 검사로 idempotent하게 막도록 구현했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/settlement/AdminAgentSettlementSnapshotController.kt`에 `POST /admin/partner/agent/settlement/finalize`를 추가하고, 인증 관리자 `member.id`를 `finalizedByMemberId`로 전달하도록 맞췄다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateService.kt`는 라이브 계산 전에 스냅샷 repository를 먼저 조회하고, 스냅샷이 있으면 generic 4종과 channel donation 1종 모두 동일 응답 DTO로 변환해 반환하도록 분기했다.
|
||||
- `docs/20260409_partner_agent_assignment_ratio_ddl.sql`에 `agent_settlement_snapshot` 테이블과 lookup/unique 인덱스 DDL을 추가했다.
|
||||
- 실행/확인 결과:
|
||||
- `lsp_diagnostics` on `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate`, `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/settlement/snapshot`, `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/settlement`, `src/test/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate`, `src/test/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/settlement` → 불가 (현재 환경에 `.kt`용 LSP 서버 미구성)
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.admin.partner.agent.settlement.*" --tests "kr.co.vividnext.sodalive.partner.agent.calculate.AgentCalculateServiceTest"` → 1차 실행 실패(신규 snapshot 도메인/서비스/분기 미구현), 구현 및 테스트 fixture 수정 후 재실행 성공
|
||||
- `./gradlew test` → 성공
|
||||
- `./gradlew build` → 1차 `ktlintTestSourceSetCheck` 실패(import 순서/unused import), 2차 `ktlintMainSourceSetCheck` 실패(import 순서/장문 line), 포맷 수정 후 재실행 성공
|
||||
|
||||
### 9차 수정
|
||||
- 무엇을: 최종 정리 과정에서 `AgentCalculateService`의 request-wide current ratio 의존성을 제거하고, 관련 테스트/빌드/수동 확인까지 다시 수행했다.
|
||||
- 왜: 시점 기준 정산 조회와 finalized snapshot-first read로 전환된 뒤에는 `ratioRepository`가 더 이상 `AgentCalculateService`에서 직접 사용되지 않으므로, 잔존 의존성을 남기지 않는 것이 실제 설계와 코드 구조를 일치시키기 때문이다.
|
||||
- 어떻게:
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateService.kt`에서 사용되지 않는 `ratioRepository` 의존성을 제거했다.
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateServiceTest.kt`, `AgentCalculateQueryRepositoryTest.kt`의 생성자 호출부를 새 시그니처에 맞게 정리했다.
|
||||
- 실행/확인 결과:
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.calculate.AgentCalculateServiceTest" --tests "kr.co.vividnext.sodalive.admin.partner.agent.settlement.*"` → 1차 실행 실패(`AgentCalculateQueryRepositoryTest`의 구 생성자 시그니처 참조), 수정 후 재실행 성공
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.calculate.AgentCalculateServiceTest" --tests "kr.co.vividnext.sodalive.partner.agent.calculate.AgentCalculateQueryRepositoryTest" --tests "kr.co.vividnext.sodalive.admin.partner.agent.settlement.*"` → 성공
|
||||
- `./gradlew test && ./gradlew build` → 성공
|
||||
- `jshell --class-path "build/classes/kotlin/main:build/resources/main:...kotlin-stdlib..."` 수동 확인 →
|
||||
- `FinalizeAgentSettlementSnapshotRequest(agentId=7, settlementType=LIVE, startDateStr=2026-02-20, endDateStr=2026-02-21).toDateRange()` 출력 확인
|
||||
- `AgentSettlementSnapshot` 1건을 `toSettlementByCreatorItems()`로 변환했을 때 `agentSettlementAmount=654` 포함 응답 아이템 출력 확인
|
||||
- `kr.co.vividnext.sodalive.admin.partner.agent.settlement.AdminAgentSettlementSnapshotController` 클래스 로딩 성공
|
||||
|
||||
### 10차 정정
|
||||
- 무엇을: 체크리스트 279번의 충족 범위를 문서/코드 기준으로 다시 대조하고, 누락된 후속 구현 항목을 기존 계획 문서에 추가했다.
|
||||
- 왜: 현재 스냅샷 구현은 `appliedAgentSettlementRatio`와 계산 숫자값은 저장하지만, 설계 초안과 체크리스트 문맥이 요구하는 `assignmentId`, `agentSettlementRatioId`는 저장하지 않아 문서 기준 완전 충족으로 보기 어려웠기 때문이다.
|
||||
- 어떻게:
|
||||
- 체크리스트 279 바로 아래에 정정 메모와 후속 체크박스 3개를 추가해 누락 범위를 `snapshot 컬럼`, `query projection`, `finalize 매핑/테스트`로 분리했다.
|
||||
- 문서 설계 초안(`assignmentId`, `agentSettlementRatioId`)과 현재 구현(`AgentSettlementSnapshot`, `AdminAgentSettlementSnapshotService`, `agent_settlement_snapshot` DDL)을 대조해 차이를 명시했다.
|
||||
- 실행/확인 결과:
|
||||
- `docs/20260408_에이전트권한및정산기능추가.md:106-123, 278-282` 확인 → 스냅샷 설계 초안과 체크리스트가 `assignmentId`, `agentSettlementRatioId`, `appliedAgentSettlementRatio`, 계산 숫자값 저장을 함께 기대함을 재확인
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/settlement/snapshot/AgentSettlementSnapshot.kt` 확인 → 현재는 `appliedAgentSettlementRatio`와 계산 숫자값만 저장하고 `assignmentId`, `agentSettlementRatioId` 필드는 없음
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/settlement/AdminAgentSettlementSnapshotService.kt` 확인 → finalize 매핑이 ratio 값과 숫자값만 채우고 FK 값은 생성하지 않음
|
||||
- `docs/20260409_partner_agent_assignment_ratio_ddl.sql` 확인 → `agent_settlement_snapshot` 테이블에도 `assignment_id`, `agent_settlement_ratio_id` 컬럼이 없음
|
||||
|
||||
### 11차 정정
|
||||
- 무엇을: ratio backdate 금지, active row 단일성 보장, 관리자 쓰기 직렬화, snapshot traceability 범위 메모를 기존 체크리스트에 후속 작업으로 추가했다.
|
||||
- 왜: 현재 `AdminAgentSettlementRatioService`는 `effectiveFrom`의 시간순 검증 없이 활성 row를 닫고 새 row를 저장해 backdate 시 interval 무결성이 깨질 수 있고, `agent_settlement_ratio`/`agent_creator_relation` DDL은 active row 중복을 막는 DB 제약이 없어 동시 요청 시 조회 결과 왜곡 위험이 있기 때문이다. 또한 snapshot의 `assignmentId`/`agentSettlementRatioId` 보완은 문서 279의 의도는 충족하지만 mixed-period creator summary의 완전 provenance까지는 아님을 분명히 할 필요가 있었다.
|
||||
- 어떻게:
|
||||
- 체크리스트 ratio 구간에 `effectiveFrom` backdate/same-time/overlap 거절 검증, `agent_settlement_ratio`/`agent_creator_relation`의 MySQL 생성 컬럼 + UNIQUE 인덱스 + 기간 무결성 제약, `MemberRepository.findByIdForUpdate(...)` 기반 비관적 락 적용 항목을 추가했다.
|
||||
- snapshot 279 보완 항목 아래에 `assignmentId`/`agentSettlementRatioId` 추가가 creator-period summary 기준 추적성은 복구하지만, mixed-period summary의 완전 provenance가 필요하면 별도 source detail 테이블이 필요하다는 참고 메모를 추가했다.
|
||||
- 실행/확인 결과:
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/ratio/AdminAgentSettlementRatioService.kt` 확인 → `effectiveFrom`의 시간순/겹침 검증 없이 active row close + insert 수행
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/assignment/AdminAgentCreatorService.kt` 확인 → overlap 검증은 있지만 비관적 락/DB active uniqueness 없이 read-then-write 수행
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/member/MemberRepository.kt`, `member/contentpreference/MemberContentPreferenceService.kt`, `member/contentpreference/MemberContentPreferenceRepository.kt` 확인 → `findByIdForUpdate`, `@Lock(PESSIMISTIC_WRITE)`, `saveAndFlush + DataIntegrityViolationException` 재조회 패턴이 저장소 기존 관례로 존재함
|
||||
- `docs/20260409_partner_agent_assignment_ratio_ddl.sql` 확인 → 현재 `agent_settlement_ratio`, `agent_creator_relation` 모두 active row 단일성 보장용 unique 제약과 기간 무결성 제약이 없음
|
||||
- MySQL 제약 조사 결과 확인 → partial unique index 대신 생성 컬럼 + UNIQUE 인덱스가 가장 실용적이며, nullable unique를 직접 사용하는 방식은 active row 중복을 막지 못함
|
||||
|
||||
### 12차 정정
|
||||
- 무엇을: 남은 후속 수정 범위를 실제 구현 순서대로 더 잘게 쪼갠 작업 계획으로 재배치했다.
|
||||
- 왜: 현재 체크리스트는 해야 할 항목은 보이지만, 어떤 순서로 진행해야 리스크가 가장 적은지와 어떤 파일부터 수정해야 하는지가 한 번에 드러나지 않았기 때문이다.
|
||||
- 어떻게:
|
||||
- `세부 구현 메모` 아래에 `### 5. 후속 보완 구현 순서 (2026-04-09 추가)` 섹션을 새로 만들고, 작업을 `ratio 입력 무결성 차단 → 관리자 쓰기 직렬화 → DDL/엔티티 정합성 점검 → snapshot traceability 연결 → 최종 회귀 검증` 순서로 나눴다.
|
||||
- 각 단계마다 실제 수정 대상 파일 경로와 테스트/검증 명령을 체크박스로 세분화해, 바로 실행 가능한 작업 순서 문서가 되도록 정리했다.
|
||||
- 실행/확인 결과:
|
||||
- `docs/20260408_에이전트권한및정산기능추가.md:326-361` 확인 → 후속 보완 구현 순서 섹션과 5개 단계 체크리스트가 문서에 반영됨
|
||||
- `grep` 확인 → `### 5. 후속 보완 구현 순서`, `MemberRepository.findByIdForUpdate(...)`, `assignmentId`, `agentSettlementRatioId`, `./gradlew test`, `./gradlew build`가 새 순서형 계획에 포함됨
|
||||
- 코드/빌드 실행 여부 → 문서 수정 단계이므로 미실행
|
||||
|
||||
### 13차 정정
|
||||
- 무엇을: snapshot traceability 문제를 `summary FK 보완 + full audit provenance detail`까지 포함해 닫는 방향으로 체크리스트를 확장했다.
|
||||
- 왜: 현재 creator-period summary snapshot은 `creatorId` 기준 병합 후 한 줄로 저장되므로, `assignmentId`/`agentSettlementRatioId`만 summary에 추가해도 mixed-period 구간의 원천 provenance는 완전히 복원되지 않기 때문이다. 완전 감사 추적이 필요하면 summary와 별도로 source detail row를 함께 저장해야 한다.
|
||||
- 어떻게:
|
||||
- snapshot 체크리스트 아래에 `agent_settlement_snapshot_source_detail`(가칭) DDL 추가, finalize의 `raw source row -> source detail 저장 -> summary 저장` 순서 보강, mixed-period summary null 규칙 테스트 고정 항목을 추가했다.
|
||||
- `세부 구현 메모 > 5-4`를 확장해 DDL → detail entity/repository → finalize 저장 순서 → summary null 규칙 → detail-summary 합계 검증까지 실제 구현 순서대로 재배치했다.
|
||||
- 실행/확인 결과:
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/settlement/AdminAgentSettlementSnapshotService.kt` 확인 → 현재는 `rows.toMergedResponseItems()`로 creator-period summary만 저장하고 source detail 저장 단계는 없음
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/GetAgentCreatorSettlementSummaryQueryData.kt`, `GetAgentChannelDonationSettlementByCreatorResponse.kt` 확인 → `creatorId` 기준 병합 구조가 mixed-period provenance 소실의 직접 원인임
|
||||
- `docs/20260408_에이전트권한및정산기능추가.md:106-123` 확인 → 문서 설계 초안은 summary FK/숫자 스냅샷까지는 기대하지만, full audit provenance는 별도 설계가 필요함
|
||||
- Oracle/탐색 결과 확인 → `assignmentId + agentSettlementRatioId` summary 보완은 부분 해결이고, `source detail`까지 포함해야 mixed-period provenance 공백이 닫힘
|
||||
|
||||
### 14차 구현 및 정리
|
||||
- 무엇을: 문서에서 미체크로 남아 있던 ratio/assignment 무결성, 쓰기 직렬화, snapshot traceability/source detail provenance, 최종 검증 항목을 실제 구현 결과에 맞게 모두 완료 처리하고, 검증 기록을 최신 상태로 갱신했다.
|
||||
- 왜: 체크리스트와 실제 코드 상태가 어긋나 있으면 이후 작업자가 범위를 잘못 이해하거나, 이미 끝난 작업을 다시 추적해야 하는 비용이 생기기 때문이다. 또한 이번 변경은 정산 무결성과 확정 스냅샷 provenance를 함께 건드렸기 때문에, 최신 검증 결과를 문서에 남겨두는 것이 필수였다.
|
||||
- 어떻게:
|
||||
- `AdminAgentSettlementRatioService`, `AdminAgentCreatorService`, `AgentSettlementSnapshot`, `AdminAgentSettlementSnapshotService`, `AgentCalculateQueryRepository`, `docs/20260409_partner_agent_assignment_ratio_ddl.sql`을 기준으로 미체크 항목을 다시 대조하고, 실제 구현된 항목은 모두 `- [x]`로 갱신했다.
|
||||
- ratio 쪽에는 `effectiveFrom` backdate/same-time/history overlap 차단과 concurrent unique violation 변환 테스트를 추가했고, assignment 쪽에는 `findByIdForUpdate(...)` + `saveAndFlush` + `DataIntegrityViolationException` 대응 패턴과 테스트를 맞췄다.
|
||||
- snapshot 쪽에는 `assignmentId`, `agentSettlementRatioId`, `agent_settlement_snapshot_source_detail` provenance 저장, mixed-period summary null 규칙, detail 합계 검증까지 반영했다.
|
||||
- 실행/확인 결과:
|
||||
- `grep "^- \[ \]|^ - \[ \]" docs/20260408_에이전트권한및정산기능추가.md` → 미체크 항목 0건 확인
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.admin.partner.agent.ratio.AgentSettlementRatioServiceTest"` → 성공
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.admin.partner.agent.settlement.AdminAgentSettlementSnapshotServiceTest"` → 성공
|
||||
- `./gradlew test` → 성공
|
||||
- `./gradlew build` → 성공
|
||||
- `Oracle` completion review → 현재 문서 기준 미체크 항목 판단을 검토한 뒤, 테스트/문서 정리를 마치면 완료 처리 가능하다는 방향 재확인
|
||||
|
||||
### 15차 수정
|
||||
- 무엇을: `agent_settlement_ratio` 이력이 없는 AGENT 정산 조회에서 agent 정산금을 0%가 아니라 10% 기본값으로 계산하도록 수정하고, 해당 동작을 서비스 테스트와 수동 계산으로 검증했다.
|
||||
- 왜: 현재 구현은 agent 비율 row가 없으면 agent 정산금이 0원으로 계산되는데, 이번 정책 결정은 미설정 상태를 10% 기본 비율로 간주하는 것이기 때문이다.
|
||||
- 어떻게:
|
||||
- `docs/20260408_에이전트권한및정산기능추가.md` 체크리스트에 기본 fallback 10% 변경 항목을 추가한 뒤 완료 처리했다.
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateServiceTest.kt`에 `agentSettlementRatio = null`일 때 일반 정산/채널후원 응답이 10%를 적용하는 RED 테스트 2건을 추가했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/GetAgentCreatorSettlementSummaryQueryData.kt`, `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/GetAgentChannelDonationSettlementByCreatorResponse.kt`에서 null fallback을 `DEFAULT_AGENT_SETTLEMENT_RATIO = 10`으로 변경했다.
|
||||
- 실행/확인 결과:
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.calculate.AgentCalculateServiceTest.shouldApplyDefaultAgentSettlementRatioWhenAgentRatioHistoryDoesNotExist" --tests "kr.co.vividnext.sodalive.partner.agent.calculate.AgentCalculateServiceTest.shouldApplyDefaultAgentSettlementRatioToChannelDonationWhenAgentRatioHistoryDoesNotExist"` → 1차 실행 실패, 수정 후 재실행 성공
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.calculate.*"` → 성공
|
||||
- `./gradlew build` → 성공
|
||||
- `lsp_diagnostics` on Kotlin changed files → 불가 (현재 환경에 `.kt`용 LSP 서버 미구성)
|
||||
- `jshell --class-path "build/classes/kotlin/main:build/resources/main:..."` 수동 확인 → `genericAgentSettlementAmount=131`, `channelAgentSettlementAmount=397`
|
||||
|
||||
### 16차 수정
|
||||
- 무엇을: 관리자 에이전트 정산 비율 생성/수정 API에 `settlementRatio` 0..100 범위 검증을 추가하고, 문서 체크리스트 미완료 항목을 실제 구현 상태로 정리했다.
|
||||
- 왜: 계획 문서에는 범위 검증 항목이 남아 있었지만 실제 서비스에는 guard가 없어 음수나 100 초과 비율이 저장될 수 있었기 때문이다.
|
||||
- 어떻게:
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/ratio/AgentSettlementRatioServiceTest.kt`에 `settlementRatio = -1` 생성, `settlementRatio = 101` 수정 요청이 `common.error.invalid_request`를 던지고 `memberRepository`/`repository`가 호출되지 않는다는 RED 테스트 2건을 추가했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/ratio/AdminAgentSettlementRatioService.kt`에 `validateSettlementRatio(settlementRatio: Int)`를 추가하고 `createAgentSettlementRatio`, `updateAgentSettlementRatio` 진입부에서 0..100 범위를 먼저 검증하도록 수정했다.
|
||||
- `docs/20260408_에이전트권한및정산기능추가.md`의 체크리스트 277 항목을 완료 처리했다.
|
||||
- 실행/확인 결과:
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.admin.partner.agent.ratio.AgentSettlementRatioServiceTest.shouldThrowWhenCreatingRatioWithSettlementRatioBelowZero" --tests "kr.co.vividnext.sodalive.admin.partner.agent.ratio.AgentSettlementRatioServiceTest.shouldThrowWhenUpdatingRatioWithSettlementRatioAboveHundred"` → 1차 실행 실패, 서비스 수정 후 재실행 성공
|
||||
- `./gradlew build` → 성공
|
||||
- `jshell --class-path "/Users/klaus/Develop/sodalive/Server/sodalive/build/classes/kotlin/main:/Users/klaus/Develop/sodalive/Server/sodalive/build/resources/main:..."` 수동 확인 → `messageKey=common.error.invalid_request`, `memberRepositoryCalls=0`, `repositoryCalls=0`
|
||||
- `lsp_diagnostics` on Kotlin changed files → 불가 (현재 환경에 `.kt`용 LSP 서버 미구성)
|
||||
|
||||
### 17차 수정
|
||||
- 무엇을: `creator/list` 현재 소속 판정을 현재 시각 활성 구간 기준으로 바로잡고, AGENT 정산 조회의 빈 페이지가 전체 결과로 새는 pagination 버그를 함께 수정했다.
|
||||
- 왜: 기존 구현은 미래 `assignedAt` 소속을 너무 일찍 노출하고 미래 `unassignedAt` 소속을 너무 일찍 숨겼으며, paged query의 사전 조회 `creatorIds`가 빈 리스트일 때 2차 rows query가 전체 결과를 다시 읽을 수 있었기 때문이다.
|
||||
- 어떻게:
|
||||
- `docs/20260408_에이전트권한및정산기능추가.md` 체크리스트에 두 버그 수정 항목을 추가한 뒤 완료 처리했다.
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateQueryRepositoryTest.kt`에 현재 시각 활성 구간 creator list 회귀 테스트와, 5개 카테고리 paged query가 빈 페이지에서 빈 rows를 반환하는 회귀 테스트를 RED로 추가했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateService.kt`에서 `getAssignedCreators()`가 `currentTime`을 한 번만 잡아 count/items 조회에 공유하도록 수정했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateQueryRepository.kt`에서 creator list 쿼리를 `assignedAt <= now < unassignedAt` 반열린 구간으로 바꾸고, 5개 paged calculate 메서드가 사전 조회 `creatorIds`가 빈 리스트면 즉시 `emptyList()`를 반환하도록 보강했다.
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateServiceTest.kt`는 변경된 repository 시그니처에 맞춰 현재 시각 인자를 검증하도록 갱신했다.
|
||||
- 실행/확인 결과:
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.calculate.AgentCalculateQueryRepositoryTest.shouldGetAssignedCreatorsOnlyWithinCurrentAssignmentWindow" --tests "kr.co.vividnext.sodalive.partner.agent.calculate.AgentCalculateQueryRepositoryTest.shouldReturnEmptyRowsWhenPagedCreatorSelectionIsEmptyAcrossAllCategories"` → 1차 실행 실패, 수정 후 재실행 성공
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.calculate.AgentCalculateQueryRepositoryTest"` → 성공
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.partner.agent.calculate.*"` → 성공
|
||||
- `./gradlew build` → 성공
|
||||
- `lsp_diagnostics` on changed Kotlin files → 불가 (현재 환경에 `.kt`용 LSP 서버 미구성)
|
||||
658
docs/plan-task/20260409_partner_agent_assignment_ratio_ddl.sql
Normal file
658
docs/plan-task/20260409_partner_agent_assignment_ratio_ddl.sql
Normal file
@@ -0,0 +1,658 @@
|
||||
SET @schema_name := DATABASE();
|
||||
|
||||
SET @agent_creator_relation_table_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.tables
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_creator_relation'
|
||||
);
|
||||
|
||||
SET @create_agent_creator_relation_table_sql := IF(
|
||||
@agent_creator_relation_table_exists = 0,
|
||||
'CREATE TABLE agent_creator_relation (
|
||||
id BIGINT NOT NULL AUTO_INCREMENT COMMENT ''PK'',
|
||||
agent_id BIGINT NOT NULL COMMENT ''에이전트 회원 ID (member.id 참조)'',
|
||||
creator_id BIGINT NOT NULL COMMENT ''크리에이터 회원 ID (member.id 참조)'',
|
||||
assigned_at TIMESTAMP NOT NULL COMMENT ''소속 시작 시각'',
|
||||
unassigned_at TIMESTAMP NULL DEFAULT NULL COMMENT ''소속 종료 시각(NULL이면 현재 활성 소속)'',
|
||||
active_creator_key TINYINT GENERATED ALWAYS AS (
|
||||
CASE
|
||||
WHEN unassigned_at IS NULL THEN 1
|
||||
ELSE NULL
|
||||
END
|
||||
) STORED,
|
||||
created_at TIMESTAMP NULL DEFAULT CURRENT_TIMESTAMP COMMENT ''생성 시각'',
|
||||
updated_at TIMESTAMP NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP COMMENT ''수정 시각'',
|
||||
PRIMARY KEY (id),
|
||||
UNIQUE KEY uk_agent_creator_relation_creator_active (creator_id, active_creator_key),
|
||||
KEY idx_agent_creator_relation_agent_id (agent_id),
|
||||
KEY idx_agent_creator_relation_creator_id (creator_id),
|
||||
KEY idx_agent_creator_relation_agent_unassigned_at (agent_id, unassigned_at),
|
||||
KEY idx_agent_creator_relation_creator_assigned_at (creator_id, assigned_at),
|
||||
KEY idx_agent_creator_relation_creator_unassigned_at (creator_id, unassigned_at),
|
||||
CONSTRAINT fk_agent_creator_relation_agent_id FOREIGN KEY (agent_id) REFERENCES member (id),
|
||||
CONSTRAINT fk_agent_creator_relation_creator_id FOREIGN KEY (creator_id) REFERENCES member (id),
|
||||
CONSTRAINT chk_agent_creator_relation_period CHECK (unassigned_at IS NULL OR assigned_at < unassigned_at)
|
||||
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COMMENT=''에이전트-크리에이터 소속 관계''',
|
||||
'SELECT ''agent_creator_relation already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE create_agent_creator_relation_table_stmt FROM @create_agent_creator_relation_table_sql;
|
||||
EXECUTE create_agent_creator_relation_table_stmt;
|
||||
DEALLOCATE PREPARE create_agent_creator_relation_table_stmt;
|
||||
|
||||
SET @agent_creator_relation_has_assigned_at := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.columns
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_creator_relation'
|
||||
AND column_name = 'assigned_at'
|
||||
);
|
||||
|
||||
SET @alter_agent_creator_relation_add_assigned_at_sql := IF(
|
||||
@agent_creator_relation_table_exists = 1 AND @agent_creator_relation_has_assigned_at = 0,
|
||||
'ALTER TABLE agent_creator_relation ADD COLUMN assigned_at TIMESTAMP NOT NULL COMMENT ''소속 시작 시각'' AFTER creator_id',
|
||||
'SELECT ''agent_creator_relation.assigned_at already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE alter_agent_creator_relation_add_assigned_at_stmt FROM @alter_agent_creator_relation_add_assigned_at_sql;
|
||||
EXECUTE alter_agent_creator_relation_add_assigned_at_stmt;
|
||||
DEALLOCATE PREPARE alter_agent_creator_relation_add_assigned_at_stmt;
|
||||
|
||||
SET @agent_creator_relation_has_unassigned_at := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.columns
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_creator_relation'
|
||||
AND column_name = 'unassigned_at'
|
||||
);
|
||||
|
||||
SET @alter_agent_creator_relation_add_unassigned_at_sql := IF(
|
||||
@agent_creator_relation_table_exists = 1 AND @agent_creator_relation_has_unassigned_at = 0,
|
||||
'ALTER TABLE agent_creator_relation ADD COLUMN unassigned_at TIMESTAMP NULL DEFAULT NULL COMMENT ''소속 종료 시각(NULL이면 현재 활성 소속)'' AFTER assigned_at',
|
||||
'SELECT ''agent_creator_relation.unassigned_at already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE alter_agent_creator_relation_add_unassigned_at_stmt FROM @alter_agent_creator_relation_add_unassigned_at_sql;
|
||||
EXECUTE alter_agent_creator_relation_add_unassigned_at_stmt;
|
||||
DEALLOCATE PREPARE alter_agent_creator_relation_add_unassigned_at_stmt;
|
||||
|
||||
SET @agent_creator_relation_has_active_creator_key := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.columns
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_creator_relation'
|
||||
AND column_name = 'active_creator_key'
|
||||
);
|
||||
|
||||
SET @alter_agent_creator_relation_add_active_creator_key_sql := IF(
|
||||
@agent_creator_relation_table_exists = 1 AND @agent_creator_relation_has_active_creator_key = 0,
|
||||
'ALTER TABLE agent_creator_relation ADD COLUMN active_creator_key TINYINT GENERATED ALWAYS AS (CASE WHEN unassigned_at IS NULL THEN 1 ELSE NULL END) STORED AFTER unassigned_at',
|
||||
'SELECT ''agent_creator_relation.active_creator_key already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE alter_agent_creator_relation_add_active_creator_key_stmt FROM @alter_agent_creator_relation_add_active_creator_key_sql;
|
||||
EXECUTE alter_agent_creator_relation_add_active_creator_key_stmt;
|
||||
DEALLOCATE PREPARE alter_agent_creator_relation_add_active_creator_key_stmt;
|
||||
|
||||
SET @agent_creator_relation_creator_unique_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.statistics
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_creator_relation'
|
||||
AND index_name = 'uk_agent_creator_relation_creator_id'
|
||||
);
|
||||
|
||||
SET @drop_agent_creator_relation_creator_unique_sql := IF(
|
||||
@agent_creator_relation_creator_unique_exists > 0,
|
||||
'ALTER TABLE agent_creator_relation DROP INDEX uk_agent_creator_relation_creator_id',
|
||||
'SELECT ''uk_agent_creator_relation_creator_id already dropped'' AS message'
|
||||
);
|
||||
|
||||
PREPARE drop_agent_creator_relation_creator_unique_stmt FROM @drop_agent_creator_relation_creator_unique_sql;
|
||||
EXECUTE drop_agent_creator_relation_creator_unique_stmt;
|
||||
DEALLOCATE PREPARE drop_agent_creator_relation_creator_unique_stmt;
|
||||
|
||||
SET @agent_creator_relation_creator_active_unique_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.statistics
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_creator_relation'
|
||||
AND index_name = 'uk_agent_creator_relation_creator_active'
|
||||
);
|
||||
|
||||
SET @add_agent_creator_relation_creator_active_unique_sql := IF(
|
||||
@agent_creator_relation_table_exists = 1 AND @agent_creator_relation_creator_active_unique_exists = 0,
|
||||
'ALTER TABLE agent_creator_relation ADD UNIQUE INDEX uk_agent_creator_relation_creator_active (creator_id, active_creator_key)',
|
||||
'SELECT ''uk_agent_creator_relation_creator_active already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_creator_relation_creator_active_unique_stmt FROM @add_agent_creator_relation_creator_active_unique_sql;
|
||||
EXECUTE add_agent_creator_relation_creator_active_unique_stmt;
|
||||
DEALLOCATE PREPARE add_agent_creator_relation_creator_active_unique_stmt;
|
||||
|
||||
SET @agent_creator_relation_period_check_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.table_constraints
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_creator_relation'
|
||||
AND constraint_name = 'chk_agent_creator_relation_period'
|
||||
AND constraint_type = 'CHECK'
|
||||
);
|
||||
|
||||
SET @add_agent_creator_relation_period_check_sql := IF(
|
||||
@agent_creator_relation_table_exists = 1 AND @agent_creator_relation_period_check_exists = 0,
|
||||
'ALTER TABLE agent_creator_relation ADD CONSTRAINT chk_agent_creator_relation_period CHECK (unassigned_at IS NULL OR assigned_at < unassigned_at)',
|
||||
'SELECT ''chk_agent_creator_relation_period already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_creator_relation_period_check_stmt FROM @add_agent_creator_relation_period_check_sql;
|
||||
EXECUTE add_agent_creator_relation_period_check_stmt;
|
||||
DEALLOCATE PREPARE add_agent_creator_relation_period_check_stmt;
|
||||
|
||||
SET @agent_creator_relation_agent_unassigned_index_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.statistics
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_creator_relation'
|
||||
AND index_name = 'idx_agent_creator_relation_agent_unassigned_at'
|
||||
);
|
||||
|
||||
SET @add_agent_creator_relation_agent_unassigned_index_sql := IF(
|
||||
@agent_creator_relation_table_exists = 1 AND @agent_creator_relation_agent_unassigned_index_exists = 0,
|
||||
'ALTER TABLE agent_creator_relation ADD INDEX idx_agent_creator_relation_agent_unassigned_at (agent_id, unassigned_at)',
|
||||
'SELECT ''idx_agent_creator_relation_agent_unassigned_at already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_creator_relation_agent_unassigned_index_stmt FROM @add_agent_creator_relation_agent_unassigned_index_sql;
|
||||
EXECUTE add_agent_creator_relation_agent_unassigned_index_stmt;
|
||||
DEALLOCATE PREPARE add_agent_creator_relation_agent_unassigned_index_stmt;
|
||||
|
||||
SET @agent_creator_relation_creator_assigned_index_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.statistics
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_creator_relation'
|
||||
AND index_name = 'idx_agent_creator_relation_creator_assigned_at'
|
||||
);
|
||||
|
||||
SET @add_agent_creator_relation_creator_assigned_index_sql := IF(
|
||||
@agent_creator_relation_table_exists = 1 AND @agent_creator_relation_creator_assigned_index_exists = 0,
|
||||
'ALTER TABLE agent_creator_relation ADD INDEX idx_agent_creator_relation_creator_assigned_at (creator_id, assigned_at)',
|
||||
'SELECT ''idx_agent_creator_relation_creator_assigned_at already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_creator_relation_creator_assigned_index_stmt FROM @add_agent_creator_relation_creator_assigned_index_sql;
|
||||
EXECUTE add_agent_creator_relation_creator_assigned_index_stmt;
|
||||
DEALLOCATE PREPARE add_agent_creator_relation_creator_assigned_index_stmt;
|
||||
|
||||
SET @agent_creator_relation_creator_unassigned_index_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.statistics
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_creator_relation'
|
||||
AND index_name = 'idx_agent_creator_relation_creator_unassigned_at'
|
||||
);
|
||||
|
||||
SET @add_agent_creator_relation_creator_unassigned_index_sql := IF(
|
||||
@agent_creator_relation_table_exists = 1 AND @agent_creator_relation_creator_unassigned_index_exists = 0,
|
||||
'ALTER TABLE agent_creator_relation ADD INDEX idx_agent_creator_relation_creator_unassigned_at (creator_id, unassigned_at)',
|
||||
'SELECT ''idx_agent_creator_relation_creator_unassigned_at already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_creator_relation_creator_unassigned_index_stmt FROM @add_agent_creator_relation_creator_unassigned_index_sql;
|
||||
EXECUTE add_agent_creator_relation_creator_unassigned_index_stmt;
|
||||
DEALLOCATE PREPARE add_agent_creator_relation_creator_unassigned_index_stmt;
|
||||
|
||||
SET @agent_settlement_ratio_table_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.tables
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_ratio'
|
||||
);
|
||||
|
||||
SET @create_agent_settlement_ratio_table_sql := IF(
|
||||
@agent_settlement_ratio_table_exists = 0,
|
||||
'CREATE TABLE agent_settlement_ratio (
|
||||
id BIGINT NOT NULL AUTO_INCREMENT COMMENT ''PK'',
|
||||
member_id BIGINT NOT NULL COMMENT ''에이전트 회원 ID (member.id 참조)'',
|
||||
settlement_ratio INT NOT NULL COMMENT ''에이전트 정산 비율(%)'',
|
||||
effective_from TIMESTAMP NOT NULL COMMENT ''비율 시작 시각'',
|
||||
effective_to TIMESTAMP NULL DEFAULT NULL COMMENT ''비율 종료 시각(NULL이면 현재 활성 비율)'',
|
||||
active_ratio_key TINYINT GENERATED ALWAYS AS (
|
||||
CASE
|
||||
WHEN effective_to IS NULL THEN 1
|
||||
ELSE NULL
|
||||
END
|
||||
) STORED,
|
||||
created_at TIMESTAMP NULL DEFAULT CURRENT_TIMESTAMP COMMENT ''생성 시각'',
|
||||
updated_at TIMESTAMP NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP COMMENT ''수정 시각'',
|
||||
PRIMARY KEY (id),
|
||||
UNIQUE KEY uk_agent_settlement_ratio_member_active (member_id, active_ratio_key),
|
||||
KEY idx_agent_settlement_ratio_member_effective_to (member_id, effective_to),
|
||||
KEY idx_agent_settlement_ratio_member_effective_from (member_id, effective_from),
|
||||
CONSTRAINT fk_agent_settlement_ratio_member_id FOREIGN KEY (member_id) REFERENCES member (id),
|
||||
CONSTRAINT chk_agent_settlement_ratio_period CHECK (effective_to IS NULL OR effective_from < effective_to)
|
||||
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COMMENT=''에이전트 정산 비율''',
|
||||
'SELECT ''agent_settlement_ratio already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE create_agent_settlement_ratio_table_stmt FROM @create_agent_settlement_ratio_table_sql;
|
||||
EXECUTE create_agent_settlement_ratio_table_stmt;
|
||||
DEALLOCATE PREPARE create_agent_settlement_ratio_table_stmt;
|
||||
|
||||
SET @agent_settlement_ratio_has_effective_from := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.columns
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_ratio'
|
||||
AND column_name = 'effective_from'
|
||||
);
|
||||
|
||||
SET @alter_agent_settlement_ratio_add_effective_from_sql := IF(
|
||||
@agent_settlement_ratio_table_exists = 1 AND @agent_settlement_ratio_has_effective_from = 0,
|
||||
'ALTER TABLE agent_settlement_ratio ADD COLUMN effective_from TIMESTAMP NOT NULL COMMENT ''비율 시작 시각'' AFTER settlement_ratio',
|
||||
'SELECT ''agent_settlement_ratio.effective_from already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE alter_agent_settlement_ratio_add_effective_from_stmt FROM @alter_agent_settlement_ratio_add_effective_from_sql;
|
||||
EXECUTE alter_agent_settlement_ratio_add_effective_from_stmt;
|
||||
DEALLOCATE PREPARE alter_agent_settlement_ratio_add_effective_from_stmt;
|
||||
|
||||
SET @agent_settlement_ratio_has_effective_to := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.columns
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_ratio'
|
||||
AND column_name = 'effective_to'
|
||||
);
|
||||
|
||||
SET @alter_agent_settlement_ratio_add_effective_to_sql := IF(
|
||||
@agent_settlement_ratio_table_exists = 1 AND @agent_settlement_ratio_has_effective_to = 0,
|
||||
'ALTER TABLE agent_settlement_ratio ADD COLUMN effective_to TIMESTAMP NULL DEFAULT NULL COMMENT ''비율 종료 시각(NULL이면 현재 활성 비율)'' AFTER effective_from',
|
||||
'SELECT ''agent_settlement_ratio.effective_to already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE alter_agent_settlement_ratio_add_effective_to_stmt FROM @alter_agent_settlement_ratio_add_effective_to_sql;
|
||||
EXECUTE alter_agent_settlement_ratio_add_effective_to_stmt;
|
||||
DEALLOCATE PREPARE alter_agent_settlement_ratio_add_effective_to_stmt;
|
||||
|
||||
SET @agent_settlement_ratio_has_active_ratio_key := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.columns
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_ratio'
|
||||
AND column_name = 'active_ratio_key'
|
||||
);
|
||||
|
||||
SET @alter_agent_settlement_ratio_add_active_ratio_key_sql := IF(
|
||||
@agent_settlement_ratio_table_exists = 1 AND @agent_settlement_ratio_has_active_ratio_key = 0,
|
||||
'ALTER TABLE agent_settlement_ratio ADD COLUMN active_ratio_key TINYINT GENERATED ALWAYS AS (CASE WHEN effective_to IS NULL THEN 1 ELSE NULL END) STORED AFTER effective_to',
|
||||
'SELECT ''agent_settlement_ratio.active_ratio_key already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE alter_agent_settlement_ratio_add_active_ratio_key_stmt FROM @alter_agent_settlement_ratio_add_active_ratio_key_sql;
|
||||
EXECUTE alter_agent_settlement_ratio_add_active_ratio_key_stmt;
|
||||
DEALLOCATE PREPARE alter_agent_settlement_ratio_add_active_ratio_key_stmt;
|
||||
|
||||
SET @agent_settlement_ratio_member_unique_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.statistics
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_ratio'
|
||||
AND index_name = 'uk_agent_settlement_ratio_member_id'
|
||||
);
|
||||
|
||||
SET @drop_agent_settlement_ratio_member_unique_sql := IF(
|
||||
@agent_settlement_ratio_member_unique_exists > 0,
|
||||
'ALTER TABLE agent_settlement_ratio DROP INDEX uk_agent_settlement_ratio_member_id',
|
||||
'SELECT ''uk_agent_settlement_ratio_member_id already dropped'' AS message'
|
||||
);
|
||||
|
||||
PREPARE drop_agent_settlement_ratio_member_unique_stmt FROM @drop_agent_settlement_ratio_member_unique_sql;
|
||||
EXECUTE drop_agent_settlement_ratio_member_unique_stmt;
|
||||
DEALLOCATE PREPARE drop_agent_settlement_ratio_member_unique_stmt;
|
||||
|
||||
SET @agent_settlement_ratio_member_active_unique_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.statistics
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_ratio'
|
||||
AND index_name = 'uk_agent_settlement_ratio_member_active'
|
||||
);
|
||||
|
||||
SET @add_agent_settlement_ratio_member_active_unique_sql := IF(
|
||||
@agent_settlement_ratio_table_exists = 1 AND @agent_settlement_ratio_member_active_unique_exists = 0,
|
||||
'ALTER TABLE agent_settlement_ratio ADD UNIQUE INDEX uk_agent_settlement_ratio_member_active (member_id, active_ratio_key)',
|
||||
'SELECT ''uk_agent_settlement_ratio_member_active already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_settlement_ratio_member_active_unique_stmt FROM @add_agent_settlement_ratio_member_active_unique_sql;
|
||||
EXECUTE add_agent_settlement_ratio_member_active_unique_stmt;
|
||||
DEALLOCATE PREPARE add_agent_settlement_ratio_member_active_unique_stmt;
|
||||
|
||||
SET @agent_settlement_ratio_has_deleted_at := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.columns
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_ratio'
|
||||
AND column_name = 'deleted_at'
|
||||
);
|
||||
|
||||
SET @drop_agent_settlement_ratio_deleted_at_sql := IF(
|
||||
@agent_settlement_ratio_table_exists = 1 AND @agent_settlement_ratio_has_deleted_at > 0,
|
||||
'ALTER TABLE agent_settlement_ratio DROP COLUMN deleted_at',
|
||||
'SELECT ''agent_settlement_ratio.deleted_at already dropped'' AS message'
|
||||
);
|
||||
|
||||
PREPARE drop_agent_settlement_ratio_deleted_at_stmt FROM @drop_agent_settlement_ratio_deleted_at_sql;
|
||||
EXECUTE drop_agent_settlement_ratio_deleted_at_stmt;
|
||||
DEALLOCATE PREPARE drop_agent_settlement_ratio_deleted_at_stmt;
|
||||
|
||||
SET @agent_settlement_ratio_period_check_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.table_constraints
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_ratio'
|
||||
AND constraint_name = 'chk_agent_settlement_ratio_period'
|
||||
AND constraint_type = 'CHECK'
|
||||
);
|
||||
|
||||
SET @add_agent_settlement_ratio_period_check_sql := IF(
|
||||
@agent_settlement_ratio_table_exists = 1 AND @agent_settlement_ratio_period_check_exists = 0,
|
||||
'ALTER TABLE agent_settlement_ratio ADD CONSTRAINT chk_agent_settlement_ratio_period CHECK (effective_to IS NULL OR effective_from < effective_to)',
|
||||
'SELECT ''chk_agent_settlement_ratio_period already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_settlement_ratio_period_check_stmt FROM @add_agent_settlement_ratio_period_check_sql;
|
||||
EXECUTE add_agent_settlement_ratio_period_check_stmt;
|
||||
DEALLOCATE PREPARE add_agent_settlement_ratio_period_check_stmt;
|
||||
|
||||
SET @agent_settlement_ratio_member_effective_to_index_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.statistics
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_ratio'
|
||||
AND index_name = 'idx_agent_settlement_ratio_member_effective_to'
|
||||
);
|
||||
|
||||
SET @add_agent_settlement_ratio_member_effective_to_index_sql := IF(
|
||||
@agent_settlement_ratio_table_exists = 1 AND @agent_settlement_ratio_member_effective_to_index_exists = 0,
|
||||
'ALTER TABLE agent_settlement_ratio ADD INDEX idx_agent_settlement_ratio_member_effective_to (member_id, effective_to)',
|
||||
'SELECT ''idx_agent_settlement_ratio_member_effective_to already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_settlement_ratio_member_effective_to_index_stmt FROM @add_agent_settlement_ratio_member_effective_to_index_sql;
|
||||
EXECUTE add_agent_settlement_ratio_member_effective_to_index_stmt;
|
||||
DEALLOCATE PREPARE add_agent_settlement_ratio_member_effective_to_index_stmt;
|
||||
|
||||
SET @agent_settlement_ratio_member_effective_from_index_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.statistics
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_ratio'
|
||||
AND index_name = 'idx_agent_settlement_ratio_member_effective_from'
|
||||
);
|
||||
|
||||
SET @add_agent_settlement_ratio_member_effective_from_index_sql := IF(
|
||||
@agent_settlement_ratio_table_exists = 1 AND @agent_settlement_ratio_member_effective_from_index_exists = 0,
|
||||
'ALTER TABLE agent_settlement_ratio ADD INDEX idx_agent_settlement_ratio_member_effective_from (member_id, effective_from)',
|
||||
'SELECT ''idx_agent_settlement_ratio_member_effective_from already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_settlement_ratio_member_effective_from_index_stmt FROM @add_agent_settlement_ratio_member_effective_from_index_sql;
|
||||
EXECUTE add_agent_settlement_ratio_member_effective_from_index_stmt;
|
||||
DEALLOCATE PREPARE add_agent_settlement_ratio_member_effective_from_index_stmt;
|
||||
|
||||
SET @agent_settlement_snapshot_table_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.tables
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_snapshot'
|
||||
);
|
||||
|
||||
SET @create_agent_settlement_snapshot_table_sql := IF(
|
||||
@agent_settlement_snapshot_table_exists = 0,
|
||||
'CREATE TABLE agent_settlement_snapshot (
|
||||
id BIGINT NOT NULL AUTO_INCREMENT COMMENT ''PK'',
|
||||
period_start TIMESTAMP NOT NULL COMMENT ''정산 기간 시작 시각'',
|
||||
period_end TIMESTAMP NOT NULL COMMENT ''정산 기간 종료 시각'',
|
||||
settlement_type VARCHAR(50) NOT NULL COMMENT ''정산 유형(LIVE, CONTENT, COMMUNITY, CHANNEL_DONATION, CONTENT_DONATION)'',
|
||||
agent_id BIGINT NOT NULL COMMENT ''에이전트 회원 ID 스냅샷'',
|
||||
agent_nickname VARCHAR(255) NOT NULL COMMENT ''에이전트 닉네임 스냅샷'',
|
||||
creator_id BIGINT NOT NULL COMMENT ''크리에이터 회원 ID 스냅샷'',
|
||||
creator_nickname VARCHAR(255) NOT NULL COMMENT ''크리에이터 닉네임 스냅샷'',
|
||||
assignment_id BIGINT NULL COMMENT ''적용된 소속 이력 row ID 스냅샷'',
|
||||
agent_settlement_ratio_id BIGINT NULL COMMENT ''적용된 에이전트 정산 비율 이력 row ID 스냅샷'',
|
||||
applied_agent_settlement_ratio INT NULL COMMENT ''적용된 에이전트 정산 비율 스냅샷(creator-level summary에 단일 값으로 귀결될 때만 저장)'',
|
||||
count INT NOT NULL COMMENT ''건수 스냅샷'',
|
||||
total_can INT NOT NULL COMMENT ''총 캔 수 스냅샷'',
|
||||
krw INT NOT NULL COMMENT ''원화 금액 스냅샷'',
|
||||
fee INT NOT NULL COMMENT ''수수료 스냅샷'',
|
||||
settlement_amount INT NOT NULL COMMENT ''크리에이터 세전 정산금 스냅샷'',
|
||||
tax INT NOT NULL COMMENT ''원천세/세금 스냅샷'',
|
||||
deposit_amount INT NOT NULL COMMENT ''입금액 스냅샷'',
|
||||
agent_settlement_amount INT NOT NULL COMMENT ''에이전트 정산금 스냅샷'',
|
||||
finalized_at TIMESTAMP NOT NULL COMMENT ''확정 시각'',
|
||||
finalized_by_member_id BIGINT NOT NULL COMMENT ''확정한 관리자 회원 ID 스냅샷'',
|
||||
created_at TIMESTAMP NULL DEFAULT CURRENT_TIMESTAMP COMMENT ''생성 시각'',
|
||||
updated_at TIMESTAMP NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP COMMENT ''수정 시각'',
|
||||
PRIMARY KEY (id),
|
||||
UNIQUE KEY uk_agent_settlement_snapshot_period_type_agent_creator (period_start, period_end, settlement_type, agent_id, creator_id),
|
||||
KEY idx_agent_settlement_snapshot_lookup (agent_id, settlement_type, period_start, period_end),
|
||||
KEY idx_agent_settlement_snapshot_creator_lookup (creator_id, settlement_type, period_start, period_end),
|
||||
KEY idx_agent_settlement_snapshot_assignment_id (assignment_id),
|
||||
KEY idx_agent_settlement_snapshot_ratio_id (agent_settlement_ratio_id),
|
||||
CONSTRAINT fk_agent_settlement_snapshot_assignment_id FOREIGN KEY (assignment_id) REFERENCES agent_creator_relation (id),
|
||||
CONSTRAINT fk_agent_settlement_snapshot_ratio_id FOREIGN KEY (agent_settlement_ratio_id) REFERENCES agent_settlement_ratio (id)
|
||||
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COMMENT=''에이전트 확정 정산 creator-level 스냅샷''',
|
||||
'SELECT ''agent_settlement_snapshot already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE create_agent_settlement_snapshot_table_stmt FROM @create_agent_settlement_snapshot_table_sql;
|
||||
EXECUTE create_agent_settlement_snapshot_table_stmt;
|
||||
DEALLOCATE PREPARE create_agent_settlement_snapshot_table_stmt;
|
||||
|
||||
SET @agent_settlement_snapshot_has_assignment_id := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.columns
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_snapshot'
|
||||
AND column_name = 'assignment_id'
|
||||
);
|
||||
|
||||
SET @alter_agent_settlement_snapshot_add_assignment_id_sql := IF(
|
||||
@agent_settlement_snapshot_table_exists = 1 AND @agent_settlement_snapshot_has_assignment_id = 0,
|
||||
'ALTER TABLE agent_settlement_snapshot ADD COLUMN assignment_id BIGINT NULL COMMENT ''적용된 소속 이력 row ID 스냅샷'' AFTER creator_nickname',
|
||||
'SELECT ''agent_settlement_snapshot.assignment_id already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE alter_agent_settlement_snapshot_add_assignment_id_stmt FROM @alter_agent_settlement_snapshot_add_assignment_id_sql;
|
||||
EXECUTE alter_agent_settlement_snapshot_add_assignment_id_stmt;
|
||||
DEALLOCATE PREPARE alter_agent_settlement_snapshot_add_assignment_id_stmt;
|
||||
|
||||
SET @agent_settlement_snapshot_has_ratio_id := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.columns
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_snapshot'
|
||||
AND column_name = 'agent_settlement_ratio_id'
|
||||
);
|
||||
|
||||
SET @alter_agent_settlement_snapshot_add_ratio_id_sql := IF(
|
||||
@agent_settlement_snapshot_table_exists = 1 AND @agent_settlement_snapshot_has_ratio_id = 0,
|
||||
'ALTER TABLE agent_settlement_snapshot ADD COLUMN agent_settlement_ratio_id BIGINT NULL COMMENT ''적용된 에이전트 정산 비율 이력 row ID 스냅샷'' AFTER assignment_id',
|
||||
'SELECT ''agent_settlement_snapshot.agent_settlement_ratio_id already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE alter_agent_settlement_snapshot_add_ratio_id_stmt FROM @alter_agent_settlement_snapshot_add_ratio_id_sql;
|
||||
EXECUTE alter_agent_settlement_snapshot_add_ratio_id_stmt;
|
||||
DEALLOCATE PREPARE alter_agent_settlement_snapshot_add_ratio_id_stmt;
|
||||
|
||||
SET @agent_settlement_snapshot_unique_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.statistics
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_snapshot'
|
||||
AND index_name = 'uk_agent_settlement_snapshot_period_type_agent_creator'
|
||||
);
|
||||
|
||||
SET @add_agent_settlement_snapshot_unique_sql := IF(
|
||||
@agent_settlement_snapshot_table_exists = 1 AND @agent_settlement_snapshot_unique_exists = 0,
|
||||
'ALTER TABLE agent_settlement_snapshot ADD UNIQUE INDEX uk_agent_settlement_snapshot_period_type_agent_creator (period_start, period_end, settlement_type, agent_id, creator_id)',
|
||||
'SELECT ''uk_agent_settlement_snapshot_period_type_agent_creator already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_settlement_snapshot_unique_stmt FROM @add_agent_settlement_snapshot_unique_sql;
|
||||
EXECUTE add_agent_settlement_snapshot_unique_stmt;
|
||||
DEALLOCATE PREPARE add_agent_settlement_snapshot_unique_stmt;
|
||||
|
||||
SET @agent_settlement_snapshot_lookup_index_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.statistics
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_snapshot'
|
||||
AND index_name = 'idx_agent_settlement_snapshot_lookup'
|
||||
);
|
||||
|
||||
SET @add_agent_settlement_snapshot_lookup_index_sql := IF(
|
||||
@agent_settlement_snapshot_table_exists = 1 AND @agent_settlement_snapshot_lookup_index_exists = 0,
|
||||
'ALTER TABLE agent_settlement_snapshot ADD INDEX idx_agent_settlement_snapshot_lookup (agent_id, settlement_type, period_start, period_end)',
|
||||
'SELECT ''idx_agent_settlement_snapshot_lookup already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_settlement_snapshot_lookup_index_stmt FROM @add_agent_settlement_snapshot_lookup_index_sql;
|
||||
EXECUTE add_agent_settlement_snapshot_lookup_index_stmt;
|
||||
DEALLOCATE PREPARE add_agent_settlement_snapshot_lookup_index_stmt;
|
||||
|
||||
SET @agent_settlement_snapshot_creator_lookup_index_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.statistics
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_snapshot'
|
||||
AND index_name = 'idx_agent_settlement_snapshot_creator_lookup'
|
||||
);
|
||||
|
||||
SET @add_agent_settlement_snapshot_creator_lookup_index_sql := IF(
|
||||
@agent_settlement_snapshot_table_exists = 1 AND @agent_settlement_snapshot_creator_lookup_index_exists = 0,
|
||||
'ALTER TABLE agent_settlement_snapshot ADD INDEX idx_agent_settlement_snapshot_creator_lookup (creator_id, settlement_type, period_start, period_end)',
|
||||
'SELECT ''idx_agent_settlement_snapshot_creator_lookup already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_settlement_snapshot_creator_lookup_index_stmt FROM @add_agent_settlement_snapshot_creator_lookup_index_sql;
|
||||
EXECUTE add_agent_settlement_snapshot_creator_lookup_index_stmt;
|
||||
DEALLOCATE PREPARE add_agent_settlement_snapshot_creator_lookup_index_stmt;
|
||||
|
||||
SET @agent_settlement_snapshot_assignment_index_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.statistics
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_snapshot'
|
||||
AND index_name = 'idx_agent_settlement_snapshot_assignment_id'
|
||||
);
|
||||
|
||||
SET @add_agent_settlement_snapshot_assignment_index_sql := IF(
|
||||
@agent_settlement_snapshot_table_exists = 1 AND @agent_settlement_snapshot_assignment_index_exists = 0,
|
||||
'ALTER TABLE agent_settlement_snapshot ADD INDEX idx_agent_settlement_snapshot_assignment_id (assignment_id)',
|
||||
'SELECT ''idx_agent_settlement_snapshot_assignment_id already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_settlement_snapshot_assignment_index_stmt FROM @add_agent_settlement_snapshot_assignment_index_sql;
|
||||
EXECUTE add_agent_settlement_snapshot_assignment_index_stmt;
|
||||
DEALLOCATE PREPARE add_agent_settlement_snapshot_assignment_index_stmt;
|
||||
|
||||
SET @agent_settlement_snapshot_ratio_index_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.statistics
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_snapshot'
|
||||
AND index_name = 'idx_agent_settlement_snapshot_ratio_id'
|
||||
);
|
||||
|
||||
SET @add_agent_settlement_snapshot_ratio_index_sql := IF(
|
||||
@agent_settlement_snapshot_table_exists = 1 AND @agent_settlement_snapshot_ratio_index_exists = 0,
|
||||
'ALTER TABLE agent_settlement_snapshot ADD INDEX idx_agent_settlement_snapshot_ratio_id (agent_settlement_ratio_id)',
|
||||
'SELECT ''idx_agent_settlement_snapshot_ratio_id already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_settlement_snapshot_ratio_index_stmt FROM @add_agent_settlement_snapshot_ratio_index_sql;
|
||||
EXECUTE add_agent_settlement_snapshot_ratio_index_stmt;
|
||||
DEALLOCATE PREPARE add_agent_settlement_snapshot_ratio_index_stmt;
|
||||
|
||||
SET @agent_settlement_snapshot_assignment_fk_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.table_constraints
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_snapshot'
|
||||
AND constraint_name = 'fk_agent_settlement_snapshot_assignment_id'
|
||||
AND constraint_type = 'FOREIGN KEY'
|
||||
);
|
||||
|
||||
SET @add_agent_settlement_snapshot_assignment_fk_sql := IF(
|
||||
@agent_settlement_snapshot_table_exists = 1 AND @agent_settlement_snapshot_assignment_fk_exists = 0,
|
||||
'ALTER TABLE agent_settlement_snapshot ADD CONSTRAINT fk_agent_settlement_snapshot_assignment_id FOREIGN KEY (assignment_id) REFERENCES agent_creator_relation (id)',
|
||||
'SELECT ''fk_agent_settlement_snapshot_assignment_id already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_settlement_snapshot_assignment_fk_stmt FROM @add_agent_settlement_snapshot_assignment_fk_sql;
|
||||
EXECUTE add_agent_settlement_snapshot_assignment_fk_stmt;
|
||||
DEALLOCATE PREPARE add_agent_settlement_snapshot_assignment_fk_stmt;
|
||||
|
||||
SET @agent_settlement_snapshot_ratio_fk_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.table_constraints
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_snapshot'
|
||||
AND constraint_name = 'fk_agent_settlement_snapshot_ratio_id'
|
||||
AND constraint_type = 'FOREIGN KEY'
|
||||
);
|
||||
|
||||
SET @add_agent_settlement_snapshot_ratio_fk_sql := IF(
|
||||
@agent_settlement_snapshot_table_exists = 1 AND @agent_settlement_snapshot_ratio_fk_exists = 0,
|
||||
'ALTER TABLE agent_settlement_snapshot ADD CONSTRAINT fk_agent_settlement_snapshot_ratio_id FOREIGN KEY (agent_settlement_ratio_id) REFERENCES agent_settlement_ratio (id)',
|
||||
'SELECT ''fk_agent_settlement_snapshot_ratio_id already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE add_agent_settlement_snapshot_ratio_fk_stmt FROM @add_agent_settlement_snapshot_ratio_fk_sql;
|
||||
EXECUTE add_agent_settlement_snapshot_ratio_fk_stmt;
|
||||
DEALLOCATE PREPARE add_agent_settlement_snapshot_ratio_fk_stmt;
|
||||
|
||||
SET @agent_settlement_snapshot_source_detail_table_exists := (
|
||||
SELECT COUNT(1)
|
||||
FROM information_schema.tables
|
||||
WHERE table_schema = @schema_name
|
||||
AND table_name = 'agent_settlement_snapshot_source_detail'
|
||||
);
|
||||
|
||||
SET @create_agent_settlement_snapshot_source_detail_table_sql := IF(
|
||||
@agent_settlement_snapshot_source_detail_table_exists = 0,
|
||||
'CREATE TABLE agent_settlement_snapshot_source_detail (
|
||||
id BIGINT NOT NULL AUTO_INCREMENT COMMENT ''PK'',
|
||||
snapshot_id BIGINT NOT NULL COMMENT ''summary snapshot FK'',
|
||||
assignment_id BIGINT NULL COMMENT ''적용된 소속 이력 row ID'',
|
||||
agent_settlement_ratio_id BIGINT NULL COMMENT ''적용된 에이전트 정산 비율 이력 row ID'',
|
||||
applied_agent_settlement_ratio INT NULL COMMENT ''적용된 에이전트 정산 비율'',
|
||||
count INT NOT NULL COMMENT ''source subtotal 건수'',
|
||||
total_can INT NOT NULL COMMENT ''source subtotal 총 캔 수'',
|
||||
krw INT NOT NULL COMMENT ''source subtotal 원화 금액'',
|
||||
fee INT NOT NULL COMMENT ''source subtotal 수수료'',
|
||||
settlement_amount INT NOT NULL COMMENT ''source subtotal 크리에이터 세전 정산금'',
|
||||
tax INT NOT NULL COMMENT ''source subtotal 세금'',
|
||||
deposit_amount INT NOT NULL COMMENT ''source subtotal 입금액'',
|
||||
agent_settlement_amount INT NOT NULL COMMENT ''source subtotal 에이전트 정산금'',
|
||||
created_at TIMESTAMP NULL DEFAULT CURRENT_TIMESTAMP COMMENT ''생성 시각'',
|
||||
updated_at TIMESTAMP NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP COMMENT ''수정 시각'',
|
||||
PRIMARY KEY (id),
|
||||
KEY idx_agent_settlement_snapshot_source_detail_snapshot_id (snapshot_id),
|
||||
KEY idx_agent_settlement_snapshot_source_detail_assignment_id (assignment_id),
|
||||
KEY idx_agent_settlement_snapshot_source_detail_ratio_id (agent_settlement_ratio_id),
|
||||
CONSTRAINT fk_agent_settlement_snapshot_source_detail_snapshot_id FOREIGN KEY (snapshot_id) REFERENCES agent_settlement_snapshot (id),
|
||||
CONSTRAINT fk_agent_settlement_snapshot_source_detail_assignment_id FOREIGN KEY (assignment_id) REFERENCES agent_creator_relation (id),
|
||||
CONSTRAINT fk_agent_settlement_snapshot_source_detail_ratio_id FOREIGN KEY (agent_settlement_ratio_id) REFERENCES agent_settlement_ratio (id)
|
||||
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COMMENT=''에이전트 확정 정산 source provenance detail''',
|
||||
'SELECT ''agent_settlement_snapshot_source_detail already exists'' AS message'
|
||||
);
|
||||
|
||||
PREPARE create_agent_settlement_snapshot_source_detail_table_stmt FROM @create_agent_settlement_snapshot_source_detail_table_sql;
|
||||
EXECUTE create_agent_settlement_snapshot_source_detail_table_stmt;
|
||||
DEALLOCATE PREPARE create_agent_settlement_snapshot_source_detail_table_stmt;
|
||||
1637
docs/plan-task/20260410_관리자에이전트정산상세조회구현계획.md
Normal file
1637
docs/plan-task/20260410_관리자에이전트정산상세조회구현계획.md
Normal file
File diff suppressed because it is too large
Load Diff
313
docs/plan-task/20260410_관리자에이전트정산상세조회설계.md
Normal file
313
docs/plan-task/20260410_관리자에이전트정산상세조회설계.md
Normal file
@@ -0,0 +1,313 @@
|
||||
# 관리자 에이전트 정산 상세 조회 설계
|
||||
|
||||
## 문서 목적
|
||||
- 관리자 페이지에서 에이전트 목록을 보고, 특정 에이전트 상세 화면에서 소속 크리에이터와 에이전트별 정산 현황을 조회할 수 있도록 백엔드 read API 설계를 고정한다.
|
||||
- 기존 `partner/agent/calculate` 계산 로직은 최대한 재사용하고, `ADMIN` 전용 조회 진입점만 별도로 추가한다.
|
||||
|
||||
## 요구사항 정리
|
||||
- 관리자 화면에는 에이전트 리스트가 필요하다.
|
||||
- 에이전트 닉네임
|
||||
- 에이전트에 속한 크리에이터 수
|
||||
- 관리자 화면에는 크리에이터 검색이 필요하다.
|
||||
- 특정 에이전트에 크리에이터를 소속시키기 위한 검색
|
||||
- 관리자 화면에는 특정 에이전트에 현재 소속된 크리에이터 목록이 필요하다.
|
||||
- 에이전트 소속 해제를 위한 목록
|
||||
- 관리자 화면에는 특정 에이전트 기준 정산 상세가 필요하다.
|
||||
- 라이브 정산 현황
|
||||
- 콘텐츠 판매 정산 현황
|
||||
- 커뮤니티 정산 현황
|
||||
- 채널 후원 정산 현황
|
||||
- 콘텐츠 후원 정산 현황
|
||||
|
||||
## 범위와 해석
|
||||
- 이번 설계는 **B안: 에이전트 목록 → 에이전트 상세** 흐름을 기준으로 한다.
|
||||
- 에이전트 목록 화면에서는 요약 정보만 제공한다.
|
||||
- 실제 정산 데이터는 에이전트 상세 화면에서 조회한다.
|
||||
- 기존 `assignment`, `ratio`, `settlement/finalize` 쓰기 기능은 유지하고, 이번 범위에서는 관리자용 read API만 추가한다.
|
||||
|
||||
## 현재 코드 기준 확인 사항
|
||||
- 관리자 전용 에이전트 관련 컨트롤러는 이미 존재한다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/assignment/AdminAgentCreatorController.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/ratio/AdminAgentSettlementRatioController.kt`
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/settlement/AdminAgentSettlementSnapshotController.kt`
|
||||
- 에이전트 본인 전용 정산 조회 컨트롤러도 이미 존재한다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateController.kt`
|
||||
- 따라서 현재 부족한 것은 정산 계산 로직이 아니라, `ADMIN`이 특정 `agentId`를 지정해 같은 데이터를 읽는 관리자 전용 read API 레이어다.
|
||||
|
||||
## 권장 아키텍처
|
||||
|
||||
### 1. 관리자 전용 read 진입점 추가
|
||||
- 신규 패키지: `kr.co.vividnext.sodalive.admin.partner.agent.read`
|
||||
- 신규 구성
|
||||
- `AdminAgentReadController`
|
||||
- `AdminAgentReadService`
|
||||
- `AdminAgentReadQueryRepository`
|
||||
- 역할
|
||||
- 에이전트 목록 조회
|
||||
- 크리에이터 검색 조회
|
||||
- 특정 에이전트 소속 크리에이터 목록 조회
|
||||
- 특정 에이전트 기준 5종 정산 조회
|
||||
|
||||
### 2. 기존 도메인 계산 로직 재사용
|
||||
- 기존 계산/집계 로직은 계속 `kr.co.vividnext.sodalive.partner.agent.calculate` 아래에 둔다.
|
||||
- 관리자용 read 서비스는 기존 `AgentCalculateService`, `AgentCalculateQueryRepository`, snapshot 조회 경로를 재사용한다.
|
||||
- 현재 `AgentCalculateService`의 정산 조회 메서드는 이미 `agentId`를 직접 받으므로, 관리자 read 서비스는 이 메서드를 그대로 호출한다.
|
||||
- 최종안은 **정산 계산 로직은 `partner.agent.calculate`에 유지하고, ADMIN은 별도 read 서비스에서 기존 agentId 기반 조회 메서드를 재사용하는 구조**다.
|
||||
|
||||
### 3. 쓰기와 읽기 축 분리
|
||||
- 쓰기 축
|
||||
- `admin.partner.agent.assignment`
|
||||
- `admin.partner.agent.ratio`
|
||||
- `admin.partner.agent.settlement`
|
||||
- 읽기 축
|
||||
- `admin.partner.agent.read`
|
||||
- `partner.agent.calculate`
|
||||
- 이렇게 나누면 “관리자 권한으로 읽는다”와 “정산 계산 규칙을 제공한다”의 책임이 섞이지 않는다.
|
||||
|
||||
## 화면 흐름 기준 API 설계
|
||||
|
||||
### 1. 에이전트 목록 API
|
||||
- 목적: 관리자 화면의 첫 진입 리스트
|
||||
- 권장 경로: `GET /admin/partner/agent/list`
|
||||
- 권한: `ADMIN`
|
||||
- 요청 파라미터
|
||||
- `pageable`
|
||||
- 응답 필드
|
||||
- `agentId`
|
||||
- `agentNickname`
|
||||
- `assignedCreatorCount`
|
||||
- `liveAgentSettlementAmount`
|
||||
- `contentAgentSettlementAmount`
|
||||
- `communityAgentSettlementAmount`
|
||||
- `contentDonationAgentSettlementAmount`
|
||||
- `channelDonationAgentSettlementAmount`
|
||||
- `totalCount`
|
||||
- 조회 기준
|
||||
- `Member.role == AGENT`
|
||||
- 현재 활성 소속 크리에이터 수만 집계
|
||||
- 활성 소속 판정은 현재 코드의 assignment window 규칙과 동일하게 현재 시각 기준 `assignedAt <= now < unassignedAt(or null)`를 따른다.
|
||||
- 정산 합계는 별도 날짜 입력 없이 **현재 월 기준**으로 계산한다.
|
||||
- 기준 시간대: `Asia/Seoul`
|
||||
- 시작 시각: 현재 월 1일 `00:00:00`
|
||||
- 종료 시각: 다음 달 1일 `00:00:00` 직전까지 포함되는 배타 상한 방식
|
||||
- 실제 조회 구간은 위 KST 월 경계를 UTC `LocalDateTime`으로 변환해 DB의 UTC 저장 시각과 비교한다.
|
||||
- 각 합계 값의 의미는 해당 기간의 상세 조회 응답 `total.agentSettlementAmount`와 동일하다.
|
||||
- 해당 월에 정산 내역이 없더라도 에이전트 목록에서는 제외하지 않고, 5종 합계를 모두 `0`으로 내려준다.
|
||||
|
||||
### 2. 크리에이터 검색 API
|
||||
- 목적: 특정 에이전트에 크리에이터를 소속시키기 전 검색
|
||||
- 권장 경로: `GET /admin/partner/agent/creator/search`
|
||||
- 권한: `ADMIN`
|
||||
- 요청 파라미터
|
||||
- `search_word`
|
||||
- `pageable`
|
||||
- 응답 필드
|
||||
- `creatorId`
|
||||
- `creatorNickname`
|
||||
- `currentAgentId` nullable
|
||||
- `currentAgentNickname` nullable
|
||||
- 동작 원칙
|
||||
- 기존 `AdminMemberController.searchCreator()` 검색 관례를 따른다.
|
||||
- 검색 결과에 현재 활성 소속 agent 정보를 붙여서, 이미 다른 agent 소속인지 운영자가 바로 판단할 수 있게 한다.
|
||||
|
||||
### 3. 특정 에이전트 소속 크리에이터 목록 API
|
||||
- 목적: 상세 화면의 소속 크리에이터 탭
|
||||
- 권장 경로: `GET /admin/partner/agent/{agentId}/creator/list`
|
||||
- 권한: `ADMIN`
|
||||
- 응답 필드
|
||||
- `creatorId`
|
||||
- `creatorNickname`
|
||||
- `assignedAt`
|
||||
- 참고
|
||||
- 현재 `GetAgentAssignedCreatorResponse`는 `creatorId`, `creatorNickname`만 제공한다.
|
||||
- 관리자 상세 화면에서는 운영 판단을 위해 `assignedAt`이 같이 내려가는 편이 자연스럽다.
|
||||
|
||||
### 4. 특정 에이전트 정산 상세 API 5종
|
||||
- 목적: 에이전트 상세 화면의 정산 탭
|
||||
- 권한: `ADMIN`
|
||||
- 권장 경로
|
||||
- `GET /admin/partner/agent/{agentId}/calculate/live-by-creator`
|
||||
- `GET /admin/partner/agent/{agentId}/calculate/content-by-creator`
|
||||
- `GET /admin/partner/agent/{agentId}/calculate/community-by-creator`
|
||||
- `GET /admin/partner/agent/{agentId}/calculate/channel-donation-by-creator`
|
||||
- `GET /admin/partner/agent/{agentId}/calculate/content-donation-by-creator`
|
||||
- 공통 요청 파라미터
|
||||
- `startDateStr`
|
||||
- `endDateStr`
|
||||
- `pageable`
|
||||
- 응답 원칙
|
||||
- 기존 AGENT 전용 응답 계약을 가능한 그대로 유지한다.
|
||||
- `totalCount`, `total`, `items` 구조 유지
|
||||
- 각 item의 집계 필드 유지
|
||||
- `count`
|
||||
- `totalCan`
|
||||
- `krw`
|
||||
- `fee`
|
||||
- `settlementAmount`
|
||||
- `agentSettlementAmount`
|
||||
- 차이점
|
||||
- 기존 AGENT API는 로그인 principal에서 `agentId`를 얻는다.
|
||||
- 관리자 API는 path variable `agentId`를 받는다.
|
||||
|
||||
## 데이터 모델/응답 설계
|
||||
|
||||
### 1. 에이전트 목록 응답 DTO
|
||||
- 신규 DTO 필요
|
||||
- DTO 명
|
||||
- `GetAdminAgentListResponse`
|
||||
- `GetAdminAgentListItem`
|
||||
- 필수 필드
|
||||
- response: `totalCount: Int`, `items: List<GetAdminAgentListItem>`
|
||||
- item:
|
||||
- `agentId: Long`
|
||||
- `agentNickname: String`
|
||||
- `assignedCreatorCount: Int`
|
||||
- `liveAgentSettlementAmount: Int`
|
||||
- `contentAgentSettlementAmount: Int`
|
||||
- `communityAgentSettlementAmount: Int`
|
||||
- `contentDonationAgentSettlementAmount: Int`
|
||||
- `channelDonationAgentSettlementAmount: Int`
|
||||
|
||||
### 2. 크리에이터 검색 응답 DTO
|
||||
- 신규 DTO 필요
|
||||
- 기존 `admin/member` 검색 응답을 그대로 쓰기보다, 현재 활성 agent 정보를 함께 주는 전용 DTO가 필요하다.
|
||||
- DTO 명
|
||||
- `SearchAdminAgentAssignableCreatorResponse`
|
||||
- `SearchAdminAgentAssignableCreatorItem`
|
||||
- 필수 필드
|
||||
- response: `totalCount: Int`, `items: List<SearchAdminAgentAssignableCreatorItem>`
|
||||
- item: `creatorId: Long`, `creatorNickname: String`, `currentAgentId: Long?`, `currentAgentNickname: String?`
|
||||
|
||||
### 3. 관리자용 소속 크리에이터 목록 DTO
|
||||
- 기존 `GetAgentAssignedCreatorResponse`를 그대로 재사용하기보다는 관리자용 항목에 `assignedAt`을 포함한 전용 DTO가 적합하다.
|
||||
- DTO 명
|
||||
- `GetAdminAgentAssignedCreatorResponse`
|
||||
- `GetAdminAgentAssignedCreatorItem`
|
||||
- 필수 필드
|
||||
- response: `totalCount: Int`, `items: List<GetAdminAgentAssignedCreatorItem>`
|
||||
- item: `creatorId: Long`, `creatorNickname: String`, `assignedAt: LocalDateTime`
|
||||
|
||||
### 4. 관리자용 정산 상세 DTO
|
||||
- 가능하면 기존 아래 DTO를 그대로 재사용한다.
|
||||
- `GetAgentSettlementByCreatorResponse`
|
||||
- `GetAgentChannelDonationSettlementByCreatorResponse`
|
||||
- 이유
|
||||
- 화면 주체만 ADMIN으로 바뀌고 데이터 shape는 동일하기 때문이다.
|
||||
- DTO까지 갈라지면 정산 계약이 중복될 가능성이 높다.
|
||||
|
||||
## 서비스 설계
|
||||
|
||||
### 1. `AdminAgentReadService`
|
||||
- 책임
|
||||
- 에이전트 목록 조회
|
||||
- 크리에이터 검색 조회
|
||||
- 에이전트 소속 크리에이터 목록 조회
|
||||
- 에이전트 정산 상세 조회 진입
|
||||
- 내부 동작
|
||||
- 목록/검색/소속 목록은 전용 read query를 사용한다.
|
||||
- 에이전트 목록 조회 시 현재 월 기준 시작/종료 시각을 서비스에서 계산해 전용 read query에 전달한다.
|
||||
- 정산 상세는 기존 `AgentCalculateService`의 agentId 기반 public 조회 메서드를 사용한다.
|
||||
|
||||
### 2. 공통 정산 read 메서드 분리
|
||||
- 현재 `AgentCalculateService`의 핵심 정산 조회 메서드는 이미 `agentId` 파라미터 중심 public 메서드다.
|
||||
- 따라서 아래 방향으로 정리한다.
|
||||
- controller는 AGENT/ADMIN 별도로 유지
|
||||
- ADMIN read 서비스는 기존 `AgentCalculateService` public 메서드를 직접 호출한다.
|
||||
- 기대 효과
|
||||
- 정산 계산 규칙 중복 제거
|
||||
- snapshot 우선 조회 / live 계산 fallback 규칙 재사용
|
||||
|
||||
## Repository / Query 방향
|
||||
|
||||
### 1. 에이전트 목록용 조회 추가
|
||||
- 필요 기능
|
||||
- AGENT role member 목록
|
||||
- 각 agent의 현재 활성 creator count
|
||||
- 각 agent의 현재 월 기준 5종 정산 합계
|
||||
- 구현 방식
|
||||
- `AdminAgentReadQueryRepository`에서 전용 projection query를 제공한다.
|
||||
- 기본 에이전트 목록과 활성 creator count는 기존 Querydsl projection query로 조회한다.
|
||||
- 현재 월 5종 정산 합계는 각 목록 item마다 기존 `AgentCalculateQueryRepository` total 조회 메서드를 재사용해 채운다.
|
||||
- 정산 row가 없는 agent도 목록에 남겨야 하므로 기존 total 조회 응답의 `0` 기본값을 그대로 사용한다.
|
||||
|
||||
### 2. 크리에이터 검색용 조회 추가
|
||||
- 필요 기능
|
||||
- creator nickname 기준 검색
|
||||
- 현재 활성 소속 agent nullable join
|
||||
- 구현 방식
|
||||
- `AdminAgentReadQueryRepository`에서 전용 검색 query를 제공한다.
|
||||
|
||||
### 3. 소속 크리에이터 목록 조회 추가
|
||||
- 기존 `AgentCalculateQueryRepository.getAssignedCreators()`를 재사용할 수 있다.
|
||||
- 다만 `assignedAt`을 응답에 내려야 하면 projection을 확장하거나 관리자 전용 projection을 추가해야 한다.
|
||||
|
||||
## 인증/예외 처리 원칙
|
||||
- 새 관리자 read 엔드포인트는 모두 `@PreAuthorize("hasRole('ADMIN')")`를 사용한다.
|
||||
- `agentId`가 존재하지 않거나 role이 `AGENT`가 아니면 `SodaException(messageKey = ...)`로 실패한다.
|
||||
- `creator/search`는 기존 `admin/member/search` 관례처럼 최소 검색어 길이 검증을 적용한다.
|
||||
- 정산 계산식, snapshot 우선 전략, assignment/ratio history 적용 규칙은 기존 `partner.agent.calculate` 동작을 그대로 사용한다.
|
||||
|
||||
## 테스트 설계
|
||||
|
||||
### 1. 컨트롤러 테스트
|
||||
- `ADMIN` 접근 성공
|
||||
- 익명 사용자 접근 실패
|
||||
- `AGENT` 또는 일반 사용자 접근 실패
|
||||
|
||||
### 2. 서비스/리포지토리 테스트
|
||||
- 에이전트 목록에서 닉네임과 현재 활성 creator count가 맞는지 검증
|
||||
- 크리에이터 검색 결과에 현재 agent 소속 정보가 올바르게 붙는지 검증
|
||||
- 특정 agent 소속 크리에이터 목록이 현재 활성 구간 기준으로만 내려오는지 검증
|
||||
|
||||
### 3. 정산 parity 테스트
|
||||
- 동일 기간, 동일 `agentId`에 대해
|
||||
- AGENT 전용 조회 응답
|
||||
- ADMIN 전용 조회 응답
|
||||
- 두 결과의 `totalCount`, `total`, `items`가 동일한지 검증
|
||||
- 대상 5종
|
||||
- live
|
||||
- content
|
||||
- community
|
||||
- channel donation
|
||||
- content donation
|
||||
|
||||
### 4. 설계 대비 구현 계획 누락 체크리스트
|
||||
- [x] 관리자 컨트롤러 테스트에 `ADMIN` 접근 성공, 익명 접근 실패, `AGENT` 또는 일반 사용자 접근 실패 시나리오가 모두 포함되어 있는지 확인한다.
|
||||
- [x] 동일 기간, 동일 `agentId` 기준으로 AGENT 전용 응답과 ADMIN 전용 응답의 `totalCount`, `total`, `items` parity를 5종 모두 검증하는 테스트가 구현 계획에 포함되어 있는지 확인한다.
|
||||
- [x] 구현 계획의 검증 기록 단계에 `무엇을/왜/어떻게`, 실제 실행 명령과 결과, 후속 수정 시 누적 기록 및 `정정` 추가 원칙이 명시되어 있는지 확인한다.
|
||||
- [x] `/admin/partner/agent/list`가 별도 날짜 입력 없이 현재 월 기준 5종 정산 합계를 계산하도록 구현 계획에 반영되어 있는지 확인한다.
|
||||
- [x] 에이전트 목록 응답 DTO와 리스트 조회 테스트에 `live/content/community/contentDonation/channelDonation` 5종 `agentSettlementAmount` summary 필드가 모두 반영되어 있는지 확인한다.
|
||||
- [x] 해당 월에 정산 내역이 없는 에이전트도 목록에서 제외하지 않고 5종 합계를 `0`으로 표기하는 쿼리/응답/테스트 계획이 포함되어 있는지 확인한다.
|
||||
- [x] 리스트 summary 값이 같은 월 기준 상세 조회 응답의 `total.agentSettlementAmount`와 일치하는지 검증하는 회귀 테스트가 구현 계획에 포함되어 있는지 확인한다.
|
||||
|
||||
## 구현 시 유의사항
|
||||
- 관리자용 정산 API를 새로 만든다고 해서 계산 query를 복제하지 않는다.
|
||||
- 현재 assignment 활성 판정은 시간창 규칙을 반드시 동일하게 사용한다.
|
||||
- 정산 응답 계약은 기존 agent 응답과 불필요하게 분기하지 않는다.
|
||||
- 목록 화면은 요약 정보만 제공하고, 상세 정산은 상세 화면에서만 조회한다.
|
||||
- 목록의 5종 합계는 상세 API의 기간 필터와 별개로, 현재 월 기준 요약값이라는 의미를 문서와 코드에서 동일하게 유지한다.
|
||||
- 해당 월에 정산 내역이 없더라도 에이전트 행은 유지하고 금액은 `0`으로 표기한다.
|
||||
|
||||
## 최종 설계 결론
|
||||
- 이번 기능은 “새 정산 엔진 추가”가 아니라 “기존 agent 정산 계산을 ADMIN에서 조회할 수 있게 read API를 보강”하는 작업으로 본다.
|
||||
- 관리자 페이지 흐름은 아래로 고정한다.
|
||||
1. `/admin/partner/agent/list`로 에이전트 목록과 현재 월 기준 5종 정산 합계 조회
|
||||
2. 상세 화면에서 `/admin/partner/agent/{agentId}/creator/list`로 현재 소속 크리에이터 조회
|
||||
3. 필요 시 `/admin/partner/agent/creator/search`로 크리에이터 검색 후 기존 assignment API로 소속 지정
|
||||
4. 상세 화면에서 `/admin/partner/agent/{agentId}/calculate/*` 5종으로 정산 현황 조회
|
||||
- 목록의 5종 합계는 상세 페이지 이동 포인트용 현재 월 summary이며, 상세 화면의 날짜 범위 조회와 역할을 구분한다.
|
||||
- 현재 월에 정산 내역이 없는 에이전트도 목록에 포함되며, 합계는 0원으로 표시한다.
|
||||
- 이 설계를 기준으로 다음 단계에서는 구현 계획 문서를 작성한다.
|
||||
|
||||
## 검증 기록
|
||||
- 1차 설계
|
||||
- 무엇을: 관리자용 에이전트 목록/상세 read API 구조와 기존 agent 계산 로직 재사용 범위를 문서로 고정했다.
|
||||
- 왜: 현재 코드에는 관리자용 assignment/ratio/finalize는 있지만, 관리자용 agent 정산 상세 조회 API는 없어 read 레이어 설계가 먼저 필요했기 때문이다.
|
||||
- 어떻게:
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/assignment/AdminAgentCreatorController.kt` 확인
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/ratio/AdminAgentSettlementRatioController.kt` 확인
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/settlement/AdminAgentSettlementSnapshotController.kt` 확인
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateController.kt` 확인
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentCalculateService.kt` 확인
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/member/AdminMemberController.kt` 확인
|
||||
- 결과: 관리자용 read API 부재와 기존 계산 로직 재사용 가능성을 문서에 반영했다.
|
||||
21
docs/plan-task/20260410_에이전트권한및정산기능컨텍스트리뷰.md
Normal file
21
docs/plan-task/20260410_에이전트권한및정산기능컨텍스트리뷰.md
Normal file
@@ -0,0 +1,21 @@
|
||||
- [x] `admin/partner/agent/**` 현재 구현과 테스트 범위 확인
|
||||
- [x] `partner/agent/**` 현재 구현과 테스트 범위 확인
|
||||
- [x] 관련 DDL/기획 문서에서 요구사항과 변경 배경 추적
|
||||
- [x] git history와 GitHub 메타데이터에서 이전 결정/경고 사항 확인
|
||||
- [x] 제외 대상 2건을 제외하고 남는 실질 이슈만 판정
|
||||
|
||||
## 검증 기록
|
||||
|
||||
### 1차 컨텍스트 리뷰
|
||||
- 무엇을: 에이전트 권한 및 정산 기능의 최근 구현/수정 이력, 현재 코드, QA 문서, 관련 테스트를 교차 검토해 남아 있는 실질 결함이 있는지 확인했다.
|
||||
- 왜: 최근 수정으로 일부 validation 이슈는 닫혔지만, event-time 이력 모델과 finalized snapshot 정책이 실제 조회 동작까지 일관되게 반영되는지 재확인이 필요했기 때문이다.
|
||||
- 어떻게:
|
||||
- `GIT_MASTER=1 git status`, `GIT_MASTER=1 git log -30 --oneline`, `GIT_MASTER=1 git log --oneline <merge-base>..HEAD`로 브랜치 상태와 최근 변경 맥락을 확인했다.
|
||||
- `docs/20260408_에이전트권한및정산기능추가.md`, `docs/20260409_partner_agent_assignment_ratio_ddl.sql`, `docs/20260410_에이전트정산기능QA.md`를 읽어 요구사항/후속 수정/제외 대상 2건을 확인했다.
|
||||
- `src/main/kotlin/kr/co/vividnext/sodalive/admin/partner/agent/**`, `src/main/kotlin/kr/co/vividnext/sodalive/partner/agent/**`, 관련 테스트 파일을 읽어 assignment, ratio, calculate, snapshot 로직을 대조했다.
|
||||
- `./gradlew test --tests "kr.co.vividnext.sodalive.admin.partner.agent.*" --tests "kr.co.vividnext.sodalive.partner.agent.calculate.*"`를 실행해 관련 테스트를 검증했다.
|
||||
- 실행/확인 결과:
|
||||
- 관련 코드/문서 검토 결과, `AgentCalculateQueryRepository.getAssignedCreatorTotalCount/getAssignedCreators`가 `assignedAt` 현재 시점 조건 없이 `unassignedAt is null`만 사용함을 확인했다.
|
||||
- `docs/20260410_에이전트정산기능QA.md:35`에 "미래 assignedAt만 가진 예약 소속은 현재 목록에 노출되지 않아야 한다" 요구사항이 명시되어 있음을 확인했다.
|
||||
- `src/test/kotlin/kr/co/vividnext/sodalive/partner/agent/calculate/AgentAssignedCreatorFutureWindowQaTest.kt`의 첫 테스트가 실제로 실패해 위 요구사항 누락이 재현됨을 확인했다.
|
||||
- `gh` 명령은 현재 환경에 설치되어 있지 않아 GitHub PR/이슈 메타데이터 조회는 불가했다 (`zsh:1: command not found: gh`).
|
||||
Some files were not shown because too many files have changed in this diff Show More
Reference in New Issue
Block a user